-
Notifications
You must be signed in to change notification settings - Fork 87
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
update upgrades.md #1712
update upgrades.md #1712
Conversation
imabdulbasit
commented
Jul 15, 2024
•
edited
Loading
edited
- This PR updates the doc/upgrades.md
- clarifies the difference between migration and version bump
- adds additional parameters for time-based upgrade
I did some rewording. I hope it is clearer this way, but might be worth getting a second opinion.
Just pushing a change seemed like the easiest way to incorporate my thoughts for improvements. Feel free to revert if you disagree. |
Looks good, thanks! |
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.
LGTM, one question, not necessarily related to this PR.