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

docs: clarify node vs npm in FAQ #1731

Merged
merged 2 commits into from
Jan 15, 2021
Merged

docs: clarify node vs npm in FAQ #1731

merged 2 commits into from
Jan 15, 2021

Conversation

vegardok
Copy link
Contributor

If you wish to only update the package.json and push via Git you can set the project to "private": true, within your package.json to prevent publishing to npm. This can be useful for using semantic-release with a non-node project.

This is still describing a node-project.

> If you wish to only update the `package.json` and push via Git you can set the project to `"private": true,` within your `package.json` to prevent publishing to [npm](https://www.npmjs.com). This can be useful for using **semantic-release** with a non-node project.

This is still describing a node-project.
@@ -8,7 +8,7 @@ By default, only the published package will contain the version, which is the on

However, the [`@semantic-release/git`](https://github.com/semantic-release/git) plugin can be used to push the updated `package.json` as well as other files to the Git repository.

If you wish to only update the `package.json` and push via Git you can set the project to `"private": true,` within your `package.json` to prevent publishing to [npm](https://www.npmjs.com). This can be useful for using **semantic-release** with a non-node project.
If you wish to only update the `package.json` and push via Git you can set the project to `"private": true,` within your `package.json` to prevent publishing to [npm](https://www.npmjs.com). This can be useful for using **semantic-release** with a non-npm project.
Copy link
Member

Choose a reason for hiding this comment

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

I think non-node is correct here. There are alternatives to the npm CLI such as yarn, but both are Node projects. The package.json file is a node convention, not an npm convention

Copy link
Member

Choose a reason for hiding this comment

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

we could rephrase

within your package.json to prevent publishing to npm

to

within your package.json to prevent publishing to the npm registry

to make it a bit more clear?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

The package.json file is a node convention, not an npm convention

Yes, exactly. The paragraph describes using package.json with { ..., "private": true, ...} so it's still definitely a node project, not a non-node project :).

My diffs intention was to differentiate between published to npm/npmjs.com vs unpublished node projects (not cli-1 vs cli-2).

within your package.json to prevent publishing to the npm registry

to make it a bit more clear?

Only if you remove the last sentence completely. I think the paragraph will be fine without it and using sematic-versioning in actual non-node projects is documented below.

@vegardok vegardok requested a review from gr2m January 14, 2021 12:41
Copy link
Member

@gr2m gr2m left a comment

Choose a reason for hiding this comment

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

great change, thank you!

@gr2m gr2m merged commit c571910 into semantic-release:master Jan 15, 2021
@semantic-release-bot
Copy link
Collaborator

🎉 This PR is included in version 17.3.3 🎉

The release is available on:

Your semantic-release bot 📦🚀

Xunnamius pushed a commit to Xunnamius/semantic-release-atam that referenced this pull request Feb 11, 2021
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

3 participants