Skip to content

Conversation

@rjpaskin
Copy link
Contributor

No description provided.

One less thing to have to fix with Rails upgrades, and gem has not
needed changes for the past few upgrades anyway.
Copilot AI review requested due to automatic review settings October 17, 2025 16:10
Copy link

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

This PR removes the upper version constraint (< 7.1) from Rails-related dependencies, allowing the gem to work with Rails 7.1 and later versions. The changes enable compatibility testing with Rails 7.0 while maintaining backward compatibility with Rails 4.2+.

Key Changes:

  • Removed upper version limits on activerecord, activesupport, and railties dependencies
  • Pinned bundler version in Docker entrypoint script
  • Added Rails 7.0 dependencies to Gemfile for testing purposes

Reviewed Changes

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

File Description
report_generator.gemspec Removed < 7.1 version constraints from Rails dependencies to allow newer versions
docker-entrypoint.sh Pinned bundler to version 2.4.22 for consistent builds
Gemfile Added Rails 7.0 gems for testing compatibility with the loosened version requirements

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

@rjpaskin rjpaskin merged commit 95b7beb into master Oct 20, 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.

4 participants