You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Suggestion: Don't create Git tags by default. Invert the logic of --no-git-tag-version. The dev should add option like --git-tag or just --tag to automatically create a tag. In my opinion we (e.g. CI) should have full control when and how to commit and tag. It's more error-prone to tag by default.
Well, there are several reasons or cases. At first we have to say that this method creates a commit and tag. And in my case I accidentally created this tags and commits. I didn't know that. I wrote and tested a CI script which executed npm version with garbage versions to test with a script that requires the version in the package.json for further changes on the repo before commit and tag. So I wanted to increase the version and further stuff before tagging. I didn't realiszed that this method created unwanted commits and tags. In my opinion this function should be explicitly enabled by the dev using the argument. In this case, there can never be cases like mine. No unintentional changes in Git before I decide to do it. ... I know this would be a breaking change for the devs who already uses it in believing to affect git. But not really critical in my opinion.
Suggestion: Don't create Git tags by default. Invert the logic of
--no-git-tag-version
. The dev should add option like--git-tag
or just--tag
to automatically create a tag. In my opinion we (e.g. CI) should have full control when and how to commit and tag. It's more error-prone to tag by default.https://docs.npmjs.com/cli/commands/npm-version
The text was updated successfully, but these errors were encountered: