Skip to content

[WIP] Drop the BUNDLED WITH pin from the jack-core generator lock#291

Draft
Tynan Burke (thearchduke) wants to merge 1 commit into
masterfrom
drop-jack-core-bundler-lock-pin
Draft

[WIP] Drop the BUNDLED WITH pin from the jack-core generator lock#291
Tynan Burke (thearchduke) wants to merge 1 commit into
masterfrom
drop-jack-core-bundler-lock-pin

Conversation

@thearchduke

Copy link
Copy Markdown
Contributor

Having trouble with the db_schemas build due to bundler versions.

The lock file currently pins bundler 1.17.3, which the jar's consumers don't have installed--db_schemas' Maven build runs Ruby 2.7 with bundler 2.1.4, and rubygems hard-errors "Could not find 'bundler' (1.17.3)" before the generator runs. If we remove these lines, bundler will use whatever version is installed.

Waiting to confirm that this is the issue and we do not need further edits to this library, via a shim in the db_schemas build.

The lock pins bundler 1.17.3, which the jar's consumers don't have
installed -- db_schemas' Maven build runs Ruby 2.7 with bundler 2.1.4, and
rubygems hard-errors "Could not find 'bundler' (1.17.3)" before the generator
runs. Without the stanza bundler uses whatever version is installed. The
bigdecimal 1.3.5 pin stays; it's the Ruby 2.7 build fix from #280.

Co-Authored-By: Claude Opus 4.8 (1M context) <[email protected]>
@coderabbitai

coderabbitai Bot commented Jul 19, 2026

Copy link
Copy Markdown

Important

Review skipped

Draft detected.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 122437d8-aa40-4501-8464-1de64a758f47

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review

Comment @coderabbitai help to get the list of available commands.

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.

1 participant