Skip to content

QUAL-2627: Add CI matrix and Mongoid 8 compatibility - #13

Merged
roloand merged 4 commits into
masterfrom
QUAL-2627-mongoidal-ci-mongoid-8
Oct 30, 2026
Merged

roloand merged 4 commits into
masterfrom
QUAL-2627-mongoidal-ci-mongoid-8

Conversation

@roloand

@roloand roloand commented Oct 30, 2026

Copy link
Copy Markdown
Collaborator

Summary

Adds GitHub Actions CI (Mongoid 7.4 / 8.0 / 8.1 x Ruby 3.0 / 3.2, against mongo:7.0) and makes the gem compatible with Mongoid 8.0 and 8.1. Version bumped to 0.3.0.

Fixes found while verifying:

  • BulkSavable#bulk_assign raised NameError when no embeds were given (typo'd EmbesMany constant, wrong namespace).
  • Nesting#nested_save! dropped keyword args on Ruby 3.

Dev dependencies loosened (mongoid >= 7.4, < 8.2; removed bundler ~> 1.6). New specs cover bulk_savable and global_id.

Why

QUAL-2627 — qualified's Rails 7 / Mongoid 8.1 upgrade needs mongoidal verified against Mongoid 8.x.

Verification

Not run locally (no MongoDB available in the environment); the CI matrix on this PR is the verification.

🤖 Generated with Claude Code

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
Comment thread gemfiles/mongoid_7_4.gemfile.lock Resolved
Comment thread spec/global_id_spec.rb Resolved
Comment thread lib/mongoidal/nesting.rb Resolved
roloand and others added 3 commits October 30, 2026 08:52
They were generated on macOS and break bundle install on the Linux CI runners.

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
json 3 rejects the quirks_mode option ActiveSupport passes to
JSON.generate, which fails the global id spec on Ruby 3.2.

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
@roloand
roloand marked this pull request as ready for review October 30, 2026 15:51
@roloand
roloand merged commit 6a5b791 into master Oct 30, 2026
6 checks passed
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