Skip to content

feat!: drop WORKSPACE mode - #349

Open
p0deje wants to merge 6 commits into
mainfrom
feat/bzlmod
Open

p0deje wants to merge 6 commits into
mainfrom
feat/bzlmod

Conversation

@p0deje

@p0deje p0deje commented Mar 1, 2026 •

Copy link
Copy Markdown
Member

This is a large chunk of work for the upcoming 1.0 release, which includes many breaking changes:

  1. WORKSPACE mode support is completely removed, and now the ruleset can only be used via Bzlmod.
  2. Default Bazel version is 9, which has no WORKSPACE mode.
  3. Removed deprecated rb_bundle repository rule, use rb_bundle_fetch instead.
  4. Added portable-ruby MRI 4.0.7 as a default toolchain.
  5. Removed MRI 3.2 from the CI matrix since it's EOL.
  6. Upgraded all known Rubies and Bundler checksums.

@p0deje p0deje changed the title feat: upgrade to bazel 9 and drop WORKSPACE mode feat!: drop WORKSPACE mode Oct 5, 2026
@p0deje
p0deje marked this pull request as ready for review October 5, 2026 15:01
@p0deje p0deje linked an issue Oct 5, 2026 that may be closed by this pull request

@sushain97 sushain97 left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

🎉

Comment thread .github/workflows/ci.yml
uses: ruby/setup-ruby@v1
with:
ruby-version: 3.4.11
ruby-version: 3.3.12

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Intentionally moving backwards?

Comment thread .github/workflows/ci.yml
uses: ruby/setup-ruby@v1
with:
ruby-version: 3.4.11
ruby-version: 3.3.12

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Same here.

Comment thread ruby/extensions.bzl
)

direct_dep_names = {name: None for name in direct_dep_names}.keys()
direct_dev_dep_names = {name: None for name in direct_dev_dep_names if name not in direct_dep_names}.keys()

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Are we not using set to maintain compat with old versions of Bazel?

Comment thread ruby/private/download.bzl
("LIBYAML_PREFIX", "libyaml", "libyaml", "PSYCH"),
("OPENSSL_PREFIX", "openssl@3", "openssl", "OPENSSL"),
]:
prefix = _execute_command(repository_ctx, ["dist/bin/ruby", "-rrbconfig", "-e", "puts ENV['%s']" % variable])

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Hm, ideally we would parallelize this…

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.

bzlmod users should get a default toolchain registration

2 participants