Rule functions should return None instead of internal Rule objects

PiperOrigin-RevId: 159438112

1 files changed, 2 insertions, 1 deletions

diff --git a/src/main/java/com/google/devtools/build/lib/rules/SkylarkRuleClassFunctions.java b/src/main/java/com/google/devtools/build/lib/rules/SkylarkRuleClassFunctions.java
index 004b215346..03e423d7db 100644
--- a/src/main/java/com/google/devtools/build/lib/rules/SkylarkRuleClassFunctions.java
+++ b/src/main/java/com/google/devtools/build/lib/rules/SkylarkRuleClassFunctions.java
@@ -764,13 +764,14 @@ public class SkylarkRuleClassFunctions {
               "Cannot instantiate a rule when loading a .bzl file. Rules can only be called from "
                   + "a BUILD file (possibly via a macro).");
         }
-        return RuleFactory.createAndAddRule(
+        RuleFactory.createAndAddRule(
             pkgContext,
             ruleClass,
             attributeValues,
             ast,
             env,
             pkgContext.getAttributeContainerFactory().apply(ruleClass));
+        return Runtime.NONE;
       } catch (InvalidRuleException | NameConflictException e) {
         throw new EvalException(ast.getLocation(), e.getMessage());
       }