Skip to content

Conversation

@danarnold
Copy link
Member

@danarnold danarnold commented Oct 2, 2025

Update ruby versions in CI and gemfiles for mongoid 8 and 9. test against ruby 3.2 for mongoid 8, and test against 3.2, 3.3, and 3.4 for mongoid 9. (ruby 3.1 is EOL)

Drop mongoid 7 from CI and from the supported versions in the gemspec, because it was causing issues and it's very old.

@danarnold danarnold changed the title Update ruby versions in CI and gemfiles Update ruby versions in CI and gemfiles, drop mongoid 7 Oct 2, 2025
@danarnold danarnold marked this pull request as ready for review October 2, 2025 21:27
@danarnold danarnold requested review from Copilot and jclusso October 2, 2025 21:27
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

Updates Ruby and Mongoid versions to modernize CI testing and dependency management by dropping support for the deprecated Mongoid 7.x and Ruby 3.1, while expanding test coverage for newer versions.

  • Updates minimum Mongoid dependency from 7.0.10 to 8.0.0 in gemspec
  • Removes mongoid7.gemfile and updates mongoid8.gemfile to use version 8.1
  • Restructures CI matrix to test Ruby 3.2 with Mongoid 8, and Ruby 3.2/3.3/3.4 with Mongoid 9

Reviewed Changes

Copilot reviewed 4 out of 7 changed files in this pull request and generated no comments.

File Description
mongoid_includes.gemspec Updates minimum Mongoid dependency to 8.0.0
gemfiles/mongoid8.gemfile Upgrades Mongoid 8 test version to 8.1
gemfiles/mongoid7.gemfile Removes entire file to drop Mongoid 7 support
.github/workflows/ci.yml Restructures CI matrix with updated Ruby/Mongoid combinations

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@danarnold danarnold merged commit f08e867 into master Oct 3, 2025
4 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.

3 participants