build(deps): bump turbo-rails from 2.0.16 to 2.0.23#991
build(deps): bump turbo-rails from 2.0.16 to 2.0.23#991dependabot[bot] wants to merge 1 commit intomasterfrom
Conversation
Bumps [turbo-rails](https://github.com/hotwired/turbo-rails) from 2.0.16 to 2.0.23. - [Release notes](https://github.com/hotwired/turbo-rails/releases) - [Commits](hotwired/turbo-rails@v2.0.16...v2.0.23) --- updated-dependencies: - dependency-name: turbo-rails dependency-version: 2.0.23 dependency-type: direct:production update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <support@github.com>
Up to standards ✅🟢 Issues
|
| Metric | Results |
|---|---|
| Complexity | 0 |
AI Reviewer: first review requested successfully. AI can make mistakes. Always validate suggestions.
TIP This summary will be updated as you push new changes.
There was a problem hiding this comment.
Pull Request Overview
The pull request successfully updates turbo-rails to 2.0.23; however, it also introduces several major transitive dependency updates that are outside the scope of this change. Specifically, the bump to minitest 6.0.6 directly contradicts the turbo-rails v2.0.21 release notes, which recommend restricting minitest to versions below 6.0. Additionally, major updates to connection_pool (v3.0) and erb (v6.0) carry breaking change risks, such as new Ruby version requirements and dependencies on the Prism parser. These should be isolated into separate PRs to ensure stability.
About this PR
- The update includes major version jumps for connection_pool (2.5 to 3.0) and erb (5.0 to 6.0). These updates are not strictly required for turbo-rails and introduce risks like the removal of ConnectionPool::Wrapper and new requirements for the Prism parser. It is recommended to revert these lockfile changes and run
bundle update turbo-railsto isolate the specific dependency.
Test suggestions
- Verify turbo-rails version update in Gemfile.lock
- Run existing test suite to verify compatibility with minitest 6.0.6
- Verify broadcast_refresh_to options forwarding behaves as expected
Prompt proposal for missing tests
Consider implementing these tests if applicable:
1. Run existing test suite to verify compatibility with minitest 6.0.6
2. Verify broadcast_refresh_to options forwarding behaves as expected
TIP Improve review quality by adding custom instructions
TIP How was this review? Give us feedback
| method_source (1.1.0) | ||
| mini_mime (1.1.5) | ||
| minitest (5.25.5) | ||
| minitest (6.0.6) |
There was a problem hiding this comment.
🟡 MEDIUM RISK
This update bumps minitest to 6.0.6, which contradicts the turbo-rails v2.0.21 release notes explicitly recommending a restriction to 'minitest < 6'. This major version jump may cause instability in the test suite or incompatibility with turbo-rails' provided test helpers. Recommendation: Revert Gemfile.lock and run bundle update turbo-rails to isolate the update to the intended gem and its necessary dependencies.
Bumps turbo-rails from 2.0.16 to 2.0.23.
Release notes
Sourced from turbo-rails's releases.
... (truncated)
Commits
435135bBump version22701f1@hotwired/turbo-railsv8.0.2327030b9Bump version52cde05@hotwired/turbo-railsv8.0.22e511fb2Bump version99dc9c5@hotwired/turbo-railsv8.0.20c2cd99fv8.0.2131c78afRespect broadcast suppressions on before/after actions (#770)16f7613Only return messages produced by block incapture_turbo_stream_broadcastsdda27a8Restrict tests tominitest<6Dependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting
@dependabot rebase.Dependabot commands and options
You can trigger Dependabot actions by commenting on this PR:
@dependabot rebasewill rebase this PR@dependabot recreatewill recreate this PR, overwriting any edits that have been made to it@dependabot show <dependency name> ignore conditionswill show all of the ignore conditions of the specified dependency@dependabot ignore this major versionwill close this PR and stop Dependabot creating any more for this major version (unless you reopen the PR or upgrade to it yourself)@dependabot ignore this minor versionwill close this PR and stop Dependabot creating any more for this minor version (unless you reopen the PR or upgrade to it yourself)@dependabot ignore this dependencywill close this PR and stop Dependabot creating any more for this dependency (unless you reopen the PR or upgrade to it yourself)