-
Notifications
You must be signed in to change notification settings - Fork 74
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
Question: default to patch release #162
Comments
Regex are not supported anymore and have been replaced by globs. See v7.0.0
|
Thanks @pvdlg will tryout with globs,
|
Hello @pvdlg
CI Logs:
|
The correct property in |
@pvdlg thank you! |
@pvdlg Definitely got a better understanding of how the configs work using 1] By default for all/any commit message should do a patch bump. 1] also works only if the commit message has the format commit-analyzer/lib/analyze-commit.js Line 24 in 595c81f
in .releaserc.json, I was hoping for any non-conventional commit message(other than Breaking: & New:) without any Below are
Logs with debug on:
|
Reading the code at fc9b28a#diff-22b2e42b5e693db528f0afe8f5c4a6fcR26 and the behavior for micromatch.isMatch(), looks like the below rule could be tweaked. Trying it out to get rid of
|
|
@pvdlg thank you again! that was it. Got it now. 🎉
|
Hi, you have a great well written tool here. am trying it out to simplify my teams release process. CI is on CircleCI. Everything worked like a charm with the default plugins.
However our team is global and our modules will have 95% patch releases. So far difficult to convince my team to bind to a commit convention.
The custom configuration I am exploring is:
1] By default for all/any commit message should do a patch bump.
2] If commit message has type:feat (or an easier identifier for feature) then minor bump.
3] if commit message has "breaking" then major bump.
Based on other closed issues here( #143, #139 ), tried this config in package.json. But this config does not honor the default all commit to patch. CI logs below: Could use some pointers.
The text was updated successfully, but these errors were encountered: