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

doc: breaking change should bump major not minor #1601

Merged
merged 1 commit into from Oct 22, 2020

Conversation

philippeboyd
Copy link
Contributor

What Changed

Change documentation for conventional-commits

Why

Breaking change indicating wrong behaviour.

Change Type

Indicate the type of change your pull request is:

  • documentation
  • patch
  • minor
  • major

According to the spec:

> BREAKING CHANGE: a commit that has a footer BREAKING CHANGE:, or appends a ! after the type/scope, introduces a breaking API change (correlating with MAJOR in semantic versioning). A BREAKING CHANGE can be part of commits of any type.
@adierkens adierkens added the documentation Changes only affect the documentation label Oct 22, 2020
@hipstersmoothie
Copy link
Collaborator

@philippeboyd Thanks!

@hipstersmoothie hipstersmoothie merged commit 9b60e1b into intuit:master Oct 22, 2020
@adierkens
Copy link
Collaborator

🚀 PR was released in v9.60.1 🚀

@adierkens adierkens added the released This issue/pull request has been released. label Oct 23, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Changes only affect the documentation released This issue/pull request has been released.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants