commit 49e755c0acbaf7bc9f6704a970980e34ce1f1819
author jcater <jcater@google.com> Mon Mar 25 10:33:18 2019 -0700
committer Copybara-Service <copybara-worker@google.com> Mon Mar 25 10:35:00 2019 -0700
tree 702fd959ac7290a4990878b624d450544c6fbbf2
parent cce2ab130b7bb1e4defe3b0f7c5dc9d694bb0573

Remove obsolete TODO: StarlarkRuleTransitionProvider and TestTrimmingTransitionFactory both require the actual Rule object.

PiperOrigin-RevId: 240169886

src/main/java/com/google/devtools/build/lib/packages/RuleTransitionFactory.java

diff --git a/src/main/java/com/google/devtools/build/lib/packages/RuleTransitionFactory.java b/src/main/java/com/google/devtools/build/lib/packages/RuleTransitionFactory.java
index f6019133..7fe079c 100644
--- a/src/main/java/com/google/devtools/build/lib/packages/RuleTransitionFactory.java
+++ b/src/main/java/com/google/devtools/build/lib/packages/RuleTransitionFactory.java
@@ -33,6 +33,7 @@
    * <p>If this class determines that no transition should be performed, it should return {@code
    * NoTransition.INSTANCE}.
    */
-  // TODO(bazel-team): refactor to only take an AttributeMap since that's how it's used anyway.
+  // TODO(bazel-team): Refactor to only take an AttributeMap. Currently the entire Rule is consumed
+  // by StarlarkRuleTransitionProvider and TestTrimmingTransitionFactory.
   PatchTransition buildTransitionFor(Rule rule);
 }