New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Fixes for Rails v7.1+ #297

Open

CodeMonkeySteve wants to merge 1 commit into NoBrainerORM:master from CodeMonkeySteve:rails-7.1

Contributor

CodeMonkeySteve commented Nov 28, 2024

Finally got around to upgrading some of my Rails apps to v7.1 and hit a couple of issues where NoBrainer code was copied from ActiveRecord.

Added convenience method for checking Rails version >= 7.1
Fix for autoload
Fix for logging in console

Not really sure how to test these, probably add a Rails 7.1 entry to the CI matrix.


          Fixes for Rails v7.1+

ed4743c

Collaborator

zedtux commented Dec 2, 2024

Thanks! I agree with you, adding Rails 7.1 to the matrix should be good enough. Let's fix the CI first.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet