Skip to content

Conversation

@mmorel-35
Copy link
Contributor

Implements automated publication to Bazel Central Registry using the publish-to-bcr tooling. s2geometry is already in BCR but publication is manual.

Changes

  • .bcr/src/ templates: metadata with maintainers, source archive config with strip_prefix pointing to src/, presubmit matching existing BCR entry
  • .bcr/config.yml: Specifies moduleRoots: ["src"] since MODULE.bazel lives in src/ directory
  • .bcr/README.md: Setup instructions for maintainers covering PAT creation, secret configuration, and troubleshooting
  • .github/workflows/publish-to-bcr.yml: Triggers on release events, uses publish-to-bcr@v1.0.0 reusable workflow with attestation support (no registry_fork parameter - auto-detects from PAT owner's fork)

Maintainer Setup Required

  1. Create Classic PAT with repo and workflow permissions
  2. PAT owner must have a fork of bazelbuild/bazel-central-registry
  3. Add PAT as BCR_PUBLISH_TOKEN repository secret

After setup, new releases automatically create BCR pull requests. Manual trigger available via workflow_dispatch.

@google-cla
Copy link

google-cla bot commented Dec 18, 2025

Thanks for your pull request! It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).

View this failed invocation of the CLA check for more information.

For the most up to date status, view the checks section at the bottom of the pull request.

Signed-off-by: Matthieu MOREL <matthieu.morel35@gmail.com>
@jmr
Copy link
Member

jmr commented Dec 18, 2025

@vtsao-openai

@jmr
Copy link
Member

jmr commented Dec 21, 2025

Looks reasonable. I need to learn more about this BCR workflow and become a maintainer of the BCR package.

@jmr jmr changed the title Add automated BCR publication via publish-to-bcr workflow Add automated publish-to-bcr workflow Dec 21, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants