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

Mention registry terms of use in manpage and registry doc #54

Merged
merged 3 commits into from Aug 18, 2018

Conversation

@kemitchell
Copy link
Contributor

@kemitchell kemitchell commented Aug 18, 2018

This small PR spruces up the language on registry terms of use in README, and adds parallel language to source files for npm-registry and man npm doc.

The general thrust is to increase the notice folks using npm get that their registry server---whoever runs it---potentially comes with terms of service they should be aware of. If there are other places where that message should appear, I'd be very interested in pushing more commits.

cc & thanks: @iarna

@kemitchell kemitchell requested a review from as a code owner Aug 18, 2018
iarna
iarna approved these changes Aug 18, 2018
@iarna iarna changed the base branch from latest to release-next Aug 18, 2018
@iarna iarna merged commit 4614826 into npm:release-next Aug 18, 2018
1 of 2 checks passed
1 of 2 checks passed
continuous-integration/travis-ci/pr The Travis CI build is in progress
Details
@travis-ci[bot]
Travis CI - Pull Request Build Passed
Details
zkat added a commit that referenced this issue Aug 20, 2018
* doc: Mention terms of use in npm-registry
* doc: Mention registry terms of use in manpage
* doc: Update terms of use language in README

Credit: @kemitchell
PR-URL: #54
@kemitchell kemitchell deleted the terms-of-use-doc branch Aug 20, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

2 participants