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

Upgrade to mongoid 9 #105

Merged
merged 4 commits into from
May 7, 2024
Merged

Conversation

sairamsrinivasan
Copy link
Contributor

PR to upgrade monogid to v9 as it was just released: https://www.mongodb.com/docs/mongoid/master/release-notes/mongoid-9.0/

@sairamsrinivasan
Copy link
Contributor Author

sairamsrinivasan commented May 5, 2024

Result of running bundle exec rake:

Running RuboCop...
Inspecting 23 files
.......................

23 files inspected, no offenses detected
/Users/saisrinivasan/.rvm/rubies/ruby-3.0.0/bin/ruby -I/Users/saisrinivasan/.rvm/gems/ruby-3.0.0/gems/rspec-core-3.13.0/lib:/Users/saisrinivasan/.rvm/gems/ruby-3.0.0/gems/rspec-support-3.13.1/lib /Users/saisrinivasan/.rvm/gems/ruby-3.0.0/gems/rspec-core-3.13.0/exe/rspec spec/mongoid-locker_spec.rb spec/test_examples_spec.rb

Randomized with seed 2142
.............................................................................................................................................................................................................................................................................

Finished in 0.28345 seconds (files took 0.35343 seconds to load)
269 examples, 0 failures

Randomized with seed 2142

Copy link
Collaborator

@dblock dblock left a comment

Choose a reason for hiding this comment

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

Glad it works out of the box! Add to CI, https://github.com/mongoid/mongoid-locker/blob/master/.github/workflows/test.yml#L28, and CHANGELOG please?

@dangerpr-bot
Copy link

dangerpr-bot commented May 5, 2024

1 Message
📖 We really appreciate pull requests that demonstrate issues, even without a fix. That said, the next step is to try and fix the failing tests!

Generated by 🚫 Danger

@sairamsrinivasan
Copy link
Contributor Author

sairamsrinivasan commented May 6, 2024

@dblock Thanks for the review. Should we release this as a MAJOR or can we keep the next version as v2.0.3?

fix rubocop

fix readme

update changelog:
'

update change log, capture release version as 2.0.3

remove line in changelog
Copy link
Collaborator

@dblock dblock left a comment

Choose a reason for hiding this comment

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

This is not a breaking change, so not 3.0, but let's bump version to 2.1.0 as part of this PR since it's a new feature.

CHANGELOG.md Outdated
* [#102](https://github.com/mongoid/mongoid-locker/pull/102): Re-added danger, removed Travis-CI config, extracted RuboCop - [@dblock](https://github.com/dblock).
* [#104](https://github.com/mongoid/mongoid-locker/pull/104): Fixed code coverage - [@dblock](https://github.com/dblock).
* [#101](https://github.com/mongoid/mongoid-locker/pull/101): Replace the $where operator with the $expr operator - [@amanfrinati](https://github.com/amanfrinati).
* Your contribution here.
* [#105](https://github.com/mongoid/mongoid-locker/pull/105): Upgraded supported mongoid version to version 9.0 - [@saisrinivasan](https://github.com/SairamSrinivasan)
Copy link
Collaborator

Choose a reason for hiding this comment

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

Change to "Added support for Mongoid 9.0". Don't forget the period at the end or danger will complain.

@dblock
Copy link
Collaborator

dblock commented May 7, 2024

Looks good, thank you. Will cut a release soon-ish unless someone beats me to it. Btw, lots of other mongoid- gems in this org could use similar help!

@dblock dblock merged commit 89b2b03 into mongoid:master May 7, 2024
19 checks passed
@dblock
Copy link
Collaborator

dblock commented May 7, 2024

I released 2.1.0.

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