Skip to content
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.

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

Fix gem version check #483

Merged
merged 2 commits into from
Nov 6, 2024
Merged

Fix gem version check #483

merged 2 commits into from
Nov 6, 2024

Conversation

ef4
Copy link
Contributor

@ef4 ef4 commented Nov 5, 2024

All the other places in graphiti that compare against ActiveRecord.version are doing it this way for compatibility with new-enough dependencies. This one seems to have been missed.

@jkeen jkeen merged commit 68e2492 into graphiti-api:master Nov 6, 2024
36 checks passed
@jkeen
Copy link
Collaborator

jkeen commented Nov 6, 2024

thanks @ef4!

github-actions bot pushed a commit that referenced this pull request Nov 6, 2024
## [1.7.6](v1.7.5...v1.7.6) (2024-11-06)

### Bug Fixes

* Gem version check ([#483](#483)) ([68e2492](68e2492))
Copy link

github-actions bot commented Nov 6, 2024

🎉 This PR is included in version 1.7.6 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants