Rewrite TODOs to use internal bug id

Issue #7010 has long been closed in OSS Bazel, but its Google-internal equivalent remains open, pending a cleanup of the depot. RELNOTES: None PiperOrigin-RevId: 305103124
bazelbuild · Apr 6, 2020 · 97f9a93 · 97f9a93
1 parent 0a89cef
commit 97f9a93
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 2 deletions.
diff --git a/src/main/java/com/google/devtools/build/lib/bazel/rules/python/BazelPyRuleClasses.java b/src/main/java/com/google/devtools/build/lib/bazel/rules/python/BazelPyRuleClasses.java
@@ -74,7 +74,7 @@ public RuleClass build(RuleClass.Builder builder, RuleDefinitionEnvironment env)
                   .mandatoryProvidersList(
                       ImmutableList.of(
                           // Legacy provider.
-                          // TODO(#7010): Remove this legacy set.
+                          // TODO(b/153363654): Remove this legacy set.
                           ImmutableList.of(
                               SkylarkProviderIdentifier.forLegacy(PyStructUtils.PROVIDER_NAME)),
                           // Modern provider.

diff --git a/src/main/java/com/google/devtools/build/lib/rules/python/PyProviderUtils.java b/src/main/java/com/google/devtools/build/lib/rules/python/PyProviderUtils.java
@@ -32,6 +32,7 @@
  *
  * <p>This class exposes a unified view over both the legacy and modern Python providers.
  */
+// TODO(b/153363654): Delete this class, go directly through PyInfo instead.
 public class PyProviderUtils {
 
   // Disable construction.

diff --git a/src/main/java/com/google/devtools/build/lib/rules/python/PyStructUtils.java b/src/main/java/com/google/devtools/build/lib/rules/python/PyStructUtils.java
@@ -28,7 +28,7 @@
 import com.google.devtools.build.lib.syntax.SkylarkType;
 
 /** Static helper class for creating and accessing instances of the legacy "py" struct provider. */
-// TODO(#7010): Remove this in favor of PyInfo.
+// TODO(b/153363654): Remove this file.
 public class PyStructUtils {
 
   // Disable construction.