-
-
Notifications
You must be signed in to change notification settings - Fork 78.9k
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
Follow up to reverted commits #11928
Conversation
…atting consistency
* Breaking backwards compatibility bumps the major | ||
* New additions without breaking backwards compatibility bumps the minor | ||
* Bug fixes and misc changes bump the patch | ||
- **Breaking backwards compatibility **bumps the major** while resetting minor and patch |
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.
Two asterisks at the beginning don't belong there.
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.
👍
* New additions without breaking backwards compatibility bumps the minor | ||
* Bug fixes and misc changes bump the patch | ||
- **Breaking backwards compatibility **bumps the major** while resetting minor and patch | ||
- New additions without breaking backwards compatibility **bumps the minor** while resetting the patch |
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.
When used as an adverb, there is never an s
in backward. Applies to new lines 142 and 143.
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.
Er, adjective.
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.
👍
🚢 🇮🇹 |
Fixes everything we broke by reverting those commits in #11927.
/cc @twbs/team