Skip to content

chore: drop overly verbose Bazel labels#10143

Open
nmattia wants to merge 2 commits intomasterfrom
nm-no-long-name
Open

chore: drop overly verbose Bazel labels#10143
nmattia wants to merge 2 commits intomasterfrom
nm-no-long-name

Conversation

@nmattia
Copy link
Copy Markdown
Contributor

@nmattia nmattia commented May 8, 2026

This uses inject_repo to shorten some Bazel labels. The labels were introduced in #7872 when rules_rust was migrated to MODULE.bazel. The workaround used here was suggested in this ticket: bazelbuild/rules_rust#2736

This uses `inject_repo` to shorten some Bazel labels. The labels were
introduced in #7872 when rules_rust
was migrated to MODULE.bazel. The workaround used here was suggested in
this ticket: bazelbuild/rules_rust#2736
@github-actions github-actions Bot added the chore label May 8, 2026
@nmattia nmattia marked this pull request as ready for review May 8, 2026 13:31
@nmattia nmattia requested a review from a team as a code owner May 8, 2026 13:31
@github-actions github-actions Bot added the @idx label May 8, 2026
Comment thread bazel/rust.MODULE.bazel
],
)

inject_repo(crate, "openssl", "lmdb", "jemalloc", "protobuf")
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
inject_repo(crate, "openssl", "lmdb", "jemalloc", "protobuf")
# inject nice names for those repos to avoid canonical naming
inject_repo(crate, "openssl", "lmdb", "jemalloc", "protobuf")

Copy link
Copy Markdown
Collaborator

@basvandijk basvandijk left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants