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

Allow version 9.x mongoid #35

Merged
merged 5 commits into from
Sep 18, 2024
Merged

Allow version 9.x mongoid #35

merged 5 commits into from
Sep 18, 2024

Conversation

chigginsboost
Copy link
Contributor

Currently, v1.0.0 only works with mongoid v9.0.0. We need to update this so consyncful works with other versions going forward.

Copy link
Member

@hapiben hapiben left a comment

Choose a reason for hiding this comment

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

🏆

@@ -41,7 +41,7 @@ Gem::Specification.new do |spec|

spec.add_dependency 'contentful', ['>=2.11.1', '<3.0.0']
spec.add_dependency 'hooks', '>=0.4.1'
spec.add_dependency 'mongoid', ['9.0.0']
spec.add_dependency 'mongoid', '~> 9'
Copy link
Member

Choose a reason for hiding this comment

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

I am wondering if we should open this up to a future version, ie >= 9

I could go either way. Seems silly to have to do this manually, on the other hand maybe its good to make sure the tests pass on a appropriate version of mongo.

@chigginsboost chigginsboost merged commit e46742a into main Sep 18, 2024
9 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