Internal change

PiperOrigin-RevId: 847925916

Author: zhangskz

.github/scripts/validate_yaml.py

@@ -1,6 +1,6 @@
 """Validate the YAML files for GitHub Actions workflows.
 
-TODO: convert to use unittest
+TODO: b/359303016 - convert to use unittest
 """
 
 import os

.github/workflows/test_cpp.yml

@@ -59,7 +59,7 @@ jobs:
           - config: { name: "aarch64", flags: "--platforms=//build_defs:aarch64_linux" }
             cache_key: aarch64-bazel8
             targets: "//src/... //src/google/protobuf/compiler:protoc_aarch64_test //third_party/utf8_range/..."
-            image: "us-docker.pkg.dev/protobuf-build/containers/test/linux/emulation:7.6.1-aarch64-f0d1e209ed9369f69d93ce418990ecff3aa08d6f" #TODO: upgrade to bazel 8
+            image: "us-docker.pkg.dev/protobuf-build/containers/test/linux/emulation:7.6.1-aarch64-f0d1e209ed9369f69d93ce418990ecff3aa08d6f" #TODO: b/409557730 - upgrade to bazel 8
     name: ${{ matrix.config.continuous-only && inputs.continuous-prefix || '' }} Linux ${{ matrix.config.name }}
     runs-on: ${{ matrix.config.runner || 'ubuntu-latest' }}
     steps:
@@ -126,7 +126,7 @@ jobs:
         id: cross-compile
         uses: protocolbuffers/protobuf-ci/cross-compile-protoc@v5
         with:
-          image: us-docker.pkg.dev/protobuf-build/containers/common/linux/bazel:7.6.1-e0df73e51131ccaf53451355d22577f377357604 #TODO: upgrade to bazel 8
+          image: us-docker.pkg.dev/protobuf-build/containers/common/linux/bazel:7.6.1-e0df73e51131ccaf53451355d22577f377357604 #TODO: b/409557730 - upgrade to bazel 8
           credentials: ${{ secrets.GAR_SERVICE_ACCOUNT }}
           architecture: linux-${{ matrix.arch }}
       - name: Setup sccache

.github/workflows/test_csharp.yml

@@ -23,7 +23,7 @@ jobs:
         with:
           ref: ${{ inputs.safe-checkout }}
 
-      # TODO(b/327464009) Run this with Bazel once codegen is handled properly.
+      # TODO Run this with Bazel once codegen is handled properly.
       - name: Run tests
         uses: protocolbuffers/protobuf-ci/docker@v5
         with:

.github/workflows/test_java.yml

@@ -31,7 +31,7 @@ jobs:
           - name: OpenJDK 8
             cache_key: '8'
             image: us-docker.pkg.dev/protobuf-build/containers/test/linux/java:8.0.1-8-b77fdae6d4771789dfc66a56bf8d806354e8011a
-            # TODO: enable the layering check. Currently it does
+            # TODO: b/318555165 - enable the layering check. Currently it does
             # not work correctly with the toolchain in this Docker image.
             targets: //java/... //java/internal:java_version --features=-layering_check
             flags: --java_language_version=8
@@ -53,7 +53,7 @@ jobs:
             image: us-docker.pkg.dev/protobuf-build/containers/common/linux/bazel:7.6.1-e0df73e51131ccaf53451355d22577f377357604
             targets: //java/... //java/internal:java_version //compatibility/...
             flags: --noenable_bzlmod
-          # TODO: restore this test once runtime uses / emulates aarch64.
+          # TODO: b/395623141 - restore this test once runtime uses / emulates aarch64.
           # - name: aarch64
           #   cache_key: 'aarch64'
           #   image: us-docker.pkg.dev/protobuf-build/containers/test/linux/emulation:7.6.1-aarch64-f0d1e209ed9369f69d93ce418990ecff3aa08d6f
@@ -76,7 +76,7 @@ jobs:
           bazel-cache: java_linux/${{ matrix.cache_key }}
           bazel: test ${{ matrix.targets }} ${{ matrix.flags }} --test_env=KOKORO_JAVA_VERSION
 
-# TODO(b/311232187) restore this test (or a better one) when gRPC has rebuilt with 26.x
+# TODO restore this test (or a better one) when gRPC has rebuilt with 26.x
   # linkage-monitor:
   #   name: Linux Linkage Monitor
   #   runs-on: ubuntu-latest
@@ -91,7 +91,7 @@ jobs:
   #         image: us-docker.pkg.dev/protobuf-build/containers/test/linux/java:8-1fdbb997433cb22c1e49ef75ad374a8d6bb88702
   #         credentials: ${{ secrets.GAR_SERVICE_ACCOUNT }}
   #         bazel-cache: java_linux/8
-  #         # TODO: enable the layering check. Currently it does
+  #         # TODO: b/318555165 - enable the layering check. Currently it does
   #         # not work correctly with the toolchain in this Docker image.
   #         bazel: test --test_output=all //java:linkage_monitor --spawn_strategy=standalone --features=-layering_check
 

.github/workflows/test_php.yml

@@ -115,7 +115,7 @@ jobs:
         id: cross-compile
         uses: protocolbuffers/protobuf-ci/cross-compile-protoc@v5
         with:
-          image: us-docker.pkg.dev/protobuf-build/containers/common/linux/bazel:7.6.1-e0df73e51131ccaf53451355d22577f377357604 #TODO: upgrade to bazel 8.0.1
+          image: us-docker.pkg.dev/protobuf-build/containers/common/linux/bazel:7.6.1-e0df73e51131ccaf53451355d22577f377357604 #TODO: b/409557730 - upgrade to bazel 8.0.1
           credentials: ${{ secrets.GAR_SERVICE_ACCOUNT }}
           architecture: linux-i386
 
@@ -147,7 +147,7 @@ jobs:
         id: cross-compile
         uses: protocolbuffers/protobuf-ci/cross-compile-protoc@v5
         with:
-          image: us-docker.pkg.dev/protobuf-build/containers/common/linux/bazel:7.6.1-e0df73e51131ccaf53451355d22577f377357604 #TODO: upgrade to bazel 8.0.1
+          image: us-docker.pkg.dev/protobuf-build/containers/common/linux/bazel:7.6.1-e0df73e51131ccaf53451355d22577f377357604 #TODO: b/409557730 - upgrade to bazel 8.0.1
           credentials: ${{ secrets.GAR_SERVICE_ACCOUNT }}
           architecture: linux-aarch64
 
@@ -205,7 +205,7 @@ jobs:
         with:
           credentials: ${{ secrets.GAR_SERVICE_ACCOUNT }}
           bazel-version: 8.0.1
-          # TODO(b/467211650) this shouldn't be necessary, remove it
+          # TODO this shouldn't be necessary, remove it
           bazel-flags: --xcode_version_config=//.github:host_xcodes_macos15
           command: |
             pushd php

.github/workflows/test_python.yml

@@ -52,7 +52,7 @@ jobs:
           - type: C++
             version: aarch64
             targets: //python/... //python:aarch64_test
-            # TODO(b/262628111) Enable this once conformance tests are fixed.
+            # TODO Enable this once conformance tests are fixed.
             flags: --define=use_fast_cpp_protos=true --test_tag_filters=-conformance
             image: us-docker.pkg.dev/protobuf-build/containers/test/linux/emulation:7.6.1-aarch64-f0d1e209ed9369f69d93ce418990ecff3aa08d6f
 

.github/workflows/test_ruby.yml

@@ -77,7 +77,7 @@ jobs:
         id: cross-compile
         uses: protocolbuffers/protobuf-ci/cross-compile-protoc@v5
         with:
-          image: us-docker.pkg.dev/protobuf-build/containers/common/linux/bazel:7.6.1-e0df73e51131ccaf53451355d22577f377357604 #TODO: upgrade to bazel 8.0.1
+          image: us-docker.pkg.dev/protobuf-build/containers/common/linux/bazel:7.6.1-e0df73e51131ccaf53451355d22577f377357604 #TODO: b/409557730 - upgrade to bazel 8.0.1
           credentials: ${{ secrets.GAR_SERVICE_ACCOUNT }}
           architecture: linux-i386
 
@@ -110,7 +110,7 @@ jobs:
         id: cross-compile
         uses: protocolbuffers/protobuf-ci/cross-compile-protoc@v5
         with:
-          image: us-docker.pkg.dev/protobuf-build/containers/common/linux/bazel:7.6.1-e0df73e51131ccaf53451355d22577f377357604 #TODO: upgrade to bazel 8.0.1
+          image: us-docker.pkg.dev/protobuf-build/containers/common/linux/bazel:7.6.1-e0df73e51131ccaf53451355d22577f377357604 #TODO: b/409557730 - upgrade to bazel 8.0.1
           credentials: ${{ secrets.GAR_SERVICE_ACCOUNT }}
           architecture: linux-aarch64
 

ci/common.bazelrc

@@ -6,7 +6,7 @@ build:dbg --compilation_mode=dbg
 build:opt --compilation_mode=opt
 
 build:san-common --config=dbg --strip=never --copt=-O1 --copt=-fno-omit-frame-pointer
-# TODO(b/392921707) ASAN hits ODR violations with shared linkage due to
+# TODO ASAN hits ODR violations with shared linkage due to
 # cc_proto_library cyclic dependency.
 build:san-common --dynamic_mode=off
 

cmake/upb_generators.cmake

@@ -18,7 +18,7 @@ foreach(generator upb upbdefs upb_minitable)
   protobuf_configure_target(protoc-gen-${generator})
   target_include_directories(protoc-gen-${generator} PRIVATE ${bootstrap_cmake_dir})
   if(protobuf_BUILD_SHARED_LIBS)
-    # TODO(b/419096242) These binaries should depend on libprotoc.
+    # TODO These binaries should depend on libprotoc.
     target_compile_definitions(protoc-gen-${generator}
       PRIVATE LIBPROTOC_EXPORTS PROTOBUF_USE_DLLS)
   endif()

conformance/testee.h

@@ -27,7 +27,7 @@
 //                .SerializeText({.print_unknown_fields = true}),
 //             ParsedPayload(EqualsProto("pb(1: "foo")pb")));
 
-// TODO(b/410123876) Possible future APIs to expand conformance coverage:
+// TODO Possible future APIs to expand conformance coverage:
 // - Add ClearUnknownFields() to InMemoryMessage
 // - Add MergeFrom() method to InMemoryMessage to merge raw binary
 // - Remove && qualifiers on Parse* and add InMemoryMessage::Merge that merges