Skip to content

Update dependencies and fix Rails 8.1 root setup - #744

Merged
tagliala merged 1 commit into
masterfrom
chore/update-dependencies
Sep 22, 2026
Merged

tagliala merged 1 commit into
masterfrom
chore/update-dependencies

Conversation

@tagliala

Copy link
Copy Markdown
Contributor

This update was produced automatically by AI.

Also fixes the Rails 8.1 root setup: #710 pointed the rails_81 CI job at the root Gemfile but never bumped it from ~> 8.0.0, so that job silently tested Rails 8.0. The root Gemfile is now ~> 8.1.0 and the lock is refreshed. (gemfiles/rails_80 still covers 8.0.)

Finally, Rails 8.1's activesupport depends on the json gem and passes a positional options hash to JSON.parse from ActiveSupport::JSON.decode, which json 3.0 rejects (fixed by rails/rails#58601, merged on 8-1-stable but unreleased). This pins json to < 3.0 and adds a dependabot ignore rule.

Commit 0f0cc56 (Test against Rails 8.1) pointed the rails_81 CI job at the root Gemfile but never bumped it from rails ~> 8.0.0, so the job silently tested Rails 8.0. Bump the root Gemfile to rails ~> 8.1.0 and refresh the lock.

Rails 8.1's activesupport depends on the json gem and passes a positional options hash to JSON.parse from ActiveSupport::JSON.decode, which json 3.0 rejects (fixed by rails/rails#58601, merged on 8-1-stable but unreleased). Pin json to 2.x and ignore json major updates in dependabot until a release includes the fix.
@tagliala
tagliala force-pushed the chore/update-dependencies branch from 787e170 to c1a026d Compare September 22, 2026 09:07
@codecov

codecov Bot commented Sep 22, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 94.33%. Comparing base (71a291f) to head (c1a026d).

Additional details and impacted files
@@           Coverage Diff           @@
##           master     #744   +/-   ##
=======================================
  Coverage   94.33%   94.33%           
=======================================
  Files          17       17           
  Lines         459      459           
=======================================
  Hits          433      433           
  Misses         26       26           

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@tagliala
tagliala merged commit 3e294d1 into master Sep 22, 2026
24 checks passed
@tagliala
tagliala deleted the chore/update-dependencies branch September 22, 2026 09: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.

1 participant