Remove return value from ctx.file_action function

It was useless anyway.

--
MOS_MIGRATED_REVID=122051899

1 files changed, 3 insertions, 3 deletions

diff --git a/src/main/java/com/google/devtools/build/lib/rules/SkylarkRuleImplementationFunctions.java b/src/main/java/com/google/devtools/build/lib/rules/SkylarkRuleImplementationFunctions.java
index ffe2fcae76..bf750445e4 100644
--- a/src/main/java/com/google/devtools/build/lib/rules/SkylarkRuleImplementationFunctions.java
+++ b/src/main/java/com/google/devtools/build/lib/rules/SkylarkRuleImplementationFunctions.java
@@ -383,7 +383,7 @@ public class SkylarkRuleImplementationFunctions {
   @SkylarkSignature(name = "file_action",
       doc = "Creates a file write action.",
       objectType = SkylarkRuleContext.class,
-      returnType = FileWriteAction.class,
+      returnType = Runtime.NoneType.class,
       mandatoryPositionals = {
         @Param(name = "self", type = SkylarkRuleContext.class, doc = "this context"),
         @Param(name = "output", type = Artifact.class, doc = "the output file"),
@@ -393,13 +393,13 @@ public class SkylarkRuleImplementationFunctions {
             doc = "whether the output file should be executable (default is False)")})
   private static final BuiltinFunction createFileWriteAction =
       new BuiltinFunction("file_action") {
-        public FileWriteAction invoke(SkylarkRuleContext ctx,
+        public Runtime.NoneType invoke(SkylarkRuleContext ctx,
             Artifact output, String content, Boolean executable)
             throws EvalException, ConversionException {
           FileWriteAction action = new FileWriteAction(
               ctx.getRuleContext().getActionOwner(), output, content, executable);
           ctx.getRuleContext().registerAction(action);
-          return action;
+          return Runtime.NONE;
         }
       };