Delete ctx.file_action

The function has been deprecated for a long time and is disabled by --incompatible_new_actions_api.

RELNOTES: None.
PiperOrigin-RevId: 318279072

src/main/java/com/google/devtools/build/lib/analysis/starlark/StarlarkRuleContext.java

diff --git a/src/main/java/com/google/devtools/build/lib/analysis/starlark/StarlarkRuleContext.java b/src/main/java/com/google/devtools/build/lib/analysis/starlark/StarlarkRuleContext.java
index 1e8de4e..7fa6f5b 100644
--- a/src/main/java/com/google/devtools/build/lib/analysis/starlark/StarlarkRuleContext.java
+++ b/src/main/java/com/google/devtools/build/lib/analysis/starlark/StarlarkRuleContext.java
@@ -76,7 +76,6 @@
 import com.google.devtools.build.lib.packages.Type.LabelClass;
 import com.google.devtools.build.lib.shell.ShellUtils;
 import com.google.devtools.build.lib.shell.ShellUtils.TokenizationException;
-import com.google.devtools.build.lib.skylarkbuildapi.FileApi;
 import com.google.devtools.build.lib.skylarkbuildapi.StarlarkRuleContextApi;
 import com.google.devtools.build.lib.syntax.ClassObject;
 import com.google.devtools.build.lib.syntax.Dict;
@@ -955,16 +954,6 @@
   }
 
   @Override
-  public NoneType fileAction(
-      FileApi output, String content, Boolean executable, StarlarkThread thread)
-      throws EvalException {
-    checkDeprecated("ctx.actions.write", "ctx.file_action", thread.getSemantics());
-    checkMutable("file_action");
-    actions().write(output, content, executable);
-    return Starlark.NONE;
-  }
-
-  @Override
   public Runfiles runfiles(
       Sequence<?> files,
       Object transitiveFiles,

src/main/java/com/google/devtools/build/lib/skylarkbuildapi/StarlarkRuleContextApi.java

diff --git a/src/main/java/com/google/devtools/build/lib/skylarkbuildapi/StarlarkRuleContextApi.java b/src/main/java/com/google/devtools/build/lib/skylarkbuildapi/StarlarkRuleContextApi.java
index 8427b22..b7b398b 100644
--- a/src/main/java/com/google/devtools/build/lib/skylarkbuildapi/StarlarkRuleContextApi.java
+++ b/src/main/java/com/google/devtools/build/lib/skylarkbuildapi/StarlarkRuleContextApi.java
@@ -717,30 +717,6 @@
       throws EvalException;
 
   @StarlarkMethod(
-      name = "file_action",
-      doc =
-          "DEPRECATED. Use <a href =\"actions.html#write\">ctx.actions.write</a> instead. <br>"
-              + "Creates a file write action.",
-      parameters = {
-        @Param(name = "output", type = FileApi.class, named = true, doc = "The output file."),
-        @Param(
-            name = "content",
-            type = String.class,
-            named = true,
-            doc = "The contents of the file."),
-        @Param(
-            name = "executable",
-            type = Boolean.class,
-            defaultValue = "False",
-            named = true,
-            doc = "Whether the output file should be executable (default is False).")
-      },
-      allowReturnNones = true,
-      useStarlarkThread = true)
-  NoneType fileAction(FileApi output, String content, Boolean executable, StarlarkThread thread)
-      throws EvalException;
-
-  @StarlarkMethod(
       name = "runfiles",
       doc = "Creates a runfiles object.",
       parameters = {

src/test/java/com/google/devtools/build/lib/skylark/StarlarkRuleContextTest.java

diff --git a/src/test/java/com/google/devtools/build/lib/skylark/StarlarkRuleContextTest.java b/src/test/java/com/google/devtools/build/lib/skylark/StarlarkRuleContextTest.java
index 32d63bf..953edf1 100644
--- a/src/test/java/com/google/devtools/build/lib/skylark/StarlarkRuleContextTest.java
+++ b/src/test/java/com/google/devtools/build/lib/skylark/StarlarkRuleContextTest.java
@@ -2398,7 +2398,6 @@
           "actions.write(file, 'foo')",
           "check_placeholders('foo', [])",
           "action(command = 'foo', outputs = [file])",
-          "file_action(file, 'foo')",
           "runfiles()",
           "resolve_command(command = 'foo')",
           "resolve_tools()");
@@ -2500,8 +2499,7 @@
       ImmutableList.of(
           "new_file('foo.txt')",
           "new_file(file, 'foo.txt')",
-          "action(command = 'foo', outputs = [file])",
-          "file_action(file, 'foo')");
+          "action(command = 'foo', outputs = [file])");
 
   @Test
   public void testIncompatibleNewActionsApi() throws Exception {