Skip to content

Rename {io_bazel_,}rules_scala_config #1709

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Feb 28, 2025

Conversation

mbland
Copy link
Contributor

@mbland mbland commented Feb 28, 2025

Description

Shortens @io_bazel_rules_scala_config to @rules_scala_config in light of the @io_bazel_rules_scala removal in #1696. Part of #1482.

Adds a section to README.md indicating this as a breaking change, with advice on working around any breakages that aren't immediately fixable.

Includes updates the "Builtin repositories no longer visible by default under Bzlmod" section to improve clarity.

Motivation

Per @simuons's advice in #1703 (comment):

@mbland I'm ok with renaming @io_bazel_rules_scala_config since there will be breaking changes as you mentioned and there is no reason for keeping label misaligned.

Shortens `@io_bazel_rules_scala_config` to `@rules_scala_config` in
light of the `@io_bazel_rules_scala` removal in bazel-contrib#1696. Part of bazel-contrib#1482.

Adds a section to `README.md` indicating this as a breaking change, with
advice on working around any breakages that aren't immediately fixable.

Includes updates the "Builtin repositories no longer visible by default
under Bzlmod" section to improve clarity.

Per @simuons's advice in
bazel-contrib#1703 (comment).
Copy link
Collaborator

@simuons simuons left a comment

Choose a reason for hiding this comment

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

Thanks, especially for documenting these changes.

@simuons simuons merged commit eadc090 into bazel-contrib:master Feb 28, 2025
2 checks passed
@mbland mbland deleted the rules-scala-config-repo-name branch February 28, 2025 19:10
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