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

Follow up to reverted commits #11928

Merged
merged 7 commits into from
Dec 18, 2013
Merged

Follow up to reverted commits #11928

merged 7 commits into from
Dec 18, 2013

Conversation

mdo
Copy link
Member

@mdo mdo commented Dec 18, 2013

Fixes everything we broke by reverting those commits in #11927.

/cc @twbs/team

* 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
Copy link
Collaborator

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.

Copy link
Member Author

Choose a reason for hiding this comment

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

👍

@ghost ghost assigned mdo Dec 18, 2013
* 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
Copy link
Collaborator

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.

Copy link
Collaborator

Choose a reason for hiding this comment

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

Er, adjective.

Copy link
Member Author

Choose a reason for hiding this comment

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

👍

@juthilo
Copy link
Collaborator

juthilo commented Dec 18, 2013

🚢 🇮🇹

mdo added a commit that referenced this pull request Dec 18, 2013
@mdo mdo merged commit af2c6de into master Dec 18, 2013
@mdo mdo deleted the we_have_to_go_back_kate branch December 18, 2013 22:49
@mdo mdo mentioned this pull request Dec 19, 2013
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants