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

fix: Recommend --tag prerelease for npm publish of prereleases #196

Merged
merged 1 commit into from
Aug 8, 2017
Merged

fix: Recommend --tag prerelease for npm publish of prereleases #196

merged 1 commit into from
Aug 8, 2017

Conversation

tmcw
Copy link
Contributor

@tmcw tmcw commented Jun 23, 2017

The --tag argument to npm publish allows you to publish a prerelease without the prerelease assuming
the 'latest' tag and causing users to download the prerelease by default. Instead, we recommend
--tag prerelease by default when --prerelease is set in the standard-version call, so to
install a prerelease version of the module, users will install module@foo or specify the exact
version of the prerelease version.

Fixes #183

Supersedes #189

The --tag argument to npm publish allows you to publish a prerelease without the prerelease assuming
the 'latest' tag and causing users to download the prerelease by default. Instead, we recommend
`--tag prerelease` by default when `--prerelease` is set in the `standard-version` call, so to
install a prerelease version of the module, users will install `module@foo` or specify the exact
version of the prerelease version.

Fixes #183
@coveralls
Copy link

coveralls commented Jun 23, 2017

Coverage Status

Coverage increased (+0.005%) to 99.519% when pulling a46a4e9 on tmcw:advise-prerelease-tag-for-npm into 2790e08 on conventional-changelog:master.

@tmcw
Copy link
Contributor Author

tmcw commented Aug 4, 2017

Ping: anything else I need to do to get this merged?

@stevemao stevemao merged commit 709dae1 into conventional-changelog:master Aug 8, 2017
@spruce
Copy link
Contributor

spruce commented Dec 3, 2017

I would be interested to have it default to the name of the release one gives on the cli. Should I create a PR?

@bcoe
Copy link
Member

bcoe commented Jan 3, 2018

@spruce definitely, this sounds like a reasonable addition to this feature.

@tmcw this work is published to npm now, sorry that took so long!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

5 participants