-
Notifications
You must be signed in to change notification settings - Fork 18
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
feat: upgrade to olive #11
Conversation
401b5d1
to
5a57cc3
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looking good. Can you please just squash your commits?
Its just two unrelated commits, one for upgrade and other docs. Do you prefer them to be combined into one |
Yes, please combine them into one. The reason is that if we merge both commits at once the v15.0.0 tag will be applied to the last one, which is the one about the maintainer. |
- Now this plugin does rely on Ruby 3 - make it clear who maintain is, Close #12
This is still work in progress.
This upgrade to ruby to 3.0.4 based on this openedx/cs_comments_service/pull/392 and openedx/cs_comments_service/pull/395