Skip to content
This repository has been archived by the owner on Nov 16, 2019. It is now read-only.

Always include NOTICE files #17

Merged
merged 1 commit into from
May 17, 2016
Merged

Always include NOTICE files #17

merged 1 commit into from
May 17, 2016

Conversation

kemitchell
Copy link
Contributor

This tiny PR adds "NOTICE" to the small category of file names that, if found, are always included in packages.

Section 4(d) of the Apache License, Version 2.0 requires distribution of licensed works with a copy of the content of the "'NOTICE' text file", if any, that comes with the project's code. Such files typically contain the name of the project and copyright notices (like "Copyright (c) 2015 Joe Schmoe") that other form licenses encourage authors to prepend to the license text itself.

A significant number of npm packages are Apache 2.0 licensed.

@zkat zkat merged commit c06bd6b into npm:master May 17, 2016
@zkat
Copy link
Contributor

zkat commented May 17, 2016

Thanks, Kyle! :)

zkat added a commit to npm/npm that referenced this pull request May 19, 2016
`fstream-npm` always includes NOTICE files now.

Credit: @kemitchell
PR-URL: npm/fstream-npm#17
zkat added a commit to npm/npm that referenced this pull request May 19, 2016
`fstream-npm` always includes NOTICE files now.

Credit: @kemitchell
PR-URL: npm/fstream-npm#17
zkat pushed a commit to npm/npm that referenced this pull request May 26, 2016
`NOTICE` files started getting included after
npm/fstream-npm#17. This documents that.

Reviewed-By: @zkat
Credit: @SimenB
PR-URL: #12782
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants