dockerfile: promote experimental linter rules

This promotes the following linter rules:
- CopyIgnoredFile
- InvalidDefinitionDescription

Signed-off-by: Jonathan A. Sternberg <[email protected]>

Author: jsternberg

frontend/dockerfile/dockerfile_lint_test.go

@@ -54,7 +54,7 @@ var lintTests = integration.TestFuncs(
 )
 
 func testDefinitionDescription(t *testing.T, sb integration.Sandbox) {
-	dockerfile := []byte(`# check=experimental=InvalidDefinitionDescription
+	dockerfile := []byte(`
 # foo this is the foo
 ARG foo=bar
 
@@ -74,7 +74,7 @@ COPY Dockerfile .
 `)
 	checkLinterWarnings(t, sb, &lintTestParams{Dockerfile: dockerfile})
 
-	dockerfile = []byte(`# check=experimental=InvalidDefinitionDescription
+	dockerfile = []byte(`
 # bar this is the bar
 ARG foo=bar
 # BasE this is the BasE image
@@ -139,7 +139,7 @@ ADD Dockerfile /windy
 		StreamBuildErrRegexp: regexp.MustCompile(`failed to solve: failed to compute cache key: failed to calculate checksum of ref [^\s]+ "/Dockerfile": not found`),
 	})
 
-	dockerfile = []byte(`# check=experimental=CopyIgnoredFile
+	dockerfile = []byte(`
 FROM scratch
 COPY Dockerfile .
 ADD Dockerfile /windy
@@ -170,7 +170,7 @@ ADD Dockerfile /windy
 		},
 	})
 
-	dockerfile = []byte(`# check=skip=all;experimental=CopyIgnoredFile
+	dockerfile = []byte(`
 FROM scratch
 COPY Dockerfile .
 ADD Dockerfile /windy

frontend/dockerfile/docs/rules/_index.md

@@ -100,11 +100,11 @@ To learn more about how to use build checks, see
       <td>FROM --platform flag should not use a constant value</td>
     </tr>
     <tr>
-      <td><a href="./copy-ignored-file/">CopyIgnoredFile (experimental)</a></td>
+      <td><a href="./copy-ignored-file/">CopyIgnoredFile</a></td>
       <td>Attempting to Copy file that is excluded by .dockerignore</td>
     </tr>
     <tr>
-      <td><a href="./invalid-definition-description/">InvalidDefinitionDescription (experimental)</a></td>
+      <td><a href="./invalid-definition-description/">InvalidDefinitionDescription</a></td>
       <td>Comment for build stage or argument should follow the format: `# <arg/stage name> <description>`. If this is not intended to be a description comment, add an empty line or comment between the instruction and the comment.</td>
     </tr>
     <tr>

frontend/dockerfile/docs/rules/copy-ignored-file.md

@@ -6,10 +6,6 @@ aliases:
   - /go/dockerfile/rule/copy-ignored-file/
 ---
 
-> [!NOTE]
-> This check is experimental and is not enabled by default. To enable it, see
-> [Experimental checks](https://docs.docker.com/go/build-checks-experimental/).
-
 ## Output
 
 ```text

frontend/dockerfile/docs/rules/invalid-definition-description.md

@@ -6,10 +6,6 @@ aliases:
   - /go/dockerfile/rule/invalid-definition-description/
 ---
 
-> [!NOTE]
-> This check is experimental and is not enabled by default. To enable it, see
-> [Experimental checks](https://docs.docker.com/go/build-checks-experimental/).
-
 ## Output
 
 ```text

frontend/dockerfile/linter/ruleset.go

@@ -163,7 +163,6 @@ var (
 		Format: func(cmd, file string) string {
 			return fmt.Sprintf("Attempting to %s file %q that is excluded by .dockerignore", cmd, file)
 		},
-		Experimental: true,
 	}
 	RuleInvalidDefinitionDescription = LinterRule[func(string, string) string]{
 		Name:        "InvalidDefinitionDescription",
@@ -172,7 +171,6 @@ var (
 		Format: func(instruction, defName string) string {
 			return fmt.Sprintf("Comment for %s should follow the format: `# %s <description>`", instruction, defName)
 		},
-		Experimental: true,
 	}
 	RuleExposeProtoCasing = LinterRule[func(string) string]{
 		Name:        "ExposeProtoCasing",