Closed G-Rath closed 4 months ago
The update enriches shakapacker
by enhancing git-based constraints handling and version formats in version_checker.rb
. It introduces new pnpm-lock.v9.yaml
files detailing dependencies and settings for peers installation and link exclusion, impacting both the library code and test specifications.
File Path | Change Summary |
---|---|
lib/shakapacker/version_checker.rb |
Added git-based version handling and adjusted key formats for "v9+" versions. |
spec/fixtures/relative_path_pnpm-lock.v9.yaml spec/fixtures/without_pnpm-lock.v9.yaml |
Introduced lockfiles with version 9.0 settings and dependencies. |
spec/shakapacker/version_checker_spec.rb |
Expanded node_package_version method to include more arguments. |
🐇✨ A hop, a skip, in the code we dip, New versions and fixes, on a digital trip. From
v9
to the stars, our packets we zip, In the warren of code, not a single slip. Cheers to changes, with a carrot tip! 🥕 🌟🐰
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media?
Note that CI is currently failing due to #470
Tests broken. Should we fix first?
An error occurred while loading ./spec/system/hello_world_spec.rb.
Failure/Error: require_relative "../config/environment"
RuntimeError:
**ERROR** Shakapacker: Shakapacker gem and node package versions do not match
Detected: 7.2.3
gem: 8.0.0.rc.2
Ensure the installed version of the gem is the same as the version of
your installed node package.
Do not use >= or ~> in your Gemfile for shakapacker without a lockfile.
Do not use ^ or ~ in your package.json for shakapacker without a lockfile.
@G-Rath please rebase on main.
@justin808 done
Summary
This was introduced with PNPN v9 and requires a couple of minor changes to support.
Pull Request checklist
Other Information
Prior art around this: https://github.com/google/osv-scanner/pull/934
Summary by CodeRabbit
New Features
Documentation
Tests
node_package_version
method for comprehensive testing.