-
Notifications
You must be signed in to change notification settings - Fork 84
Prepare for 2.0.0 release #89
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
base: master
Are you sure you want to change the base?
Conversation
RELEASING.md
Outdated
git push github master | ||
``` | ||
|
||
Release. |
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.
Suggestion: We might want to add that when releasing, create a GitHub release containing release notes too? (this might be done already by the rake release and I'm not well informed)
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.
Suggestion (optional): The only other thing we might want to add is to setup MFA to be required for rubygems publishing - https://guides.rubygems.org/mfa-requirement-opt-in/ - to reduce the risk of an account compromise leading to dodgy stuff being published.
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.
Yes to both. @aserafin I'm unsure whether you need to enable mandatory MFA on the rubygems side.
And I still need to nab the release rake task from grape-active_model_serializers which is where the contributing and releasing docs are from.
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.
Apparently this is a feature of rubygems itself and we get it by requiring gem_tasks in our Rakefile. I haven't maintained many gems and I had no idea this existed 😆 https://schneems.com/blogs/2016-03-18-bundler-release-tasks
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.
Added in c147718 with a rake task to do the whole shebang too.
@samsonjs I added you as owner to the gem 🥳 you should receive invitation email 👍 |
Thanks so much @aserafin I'm in! I'll try to release 2.0 in the next week or two. |
Added a changelog. I'm still waiting on rubygems access to do the release.