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

Add verbatim LICENSE #6

Merged
merged 1 commit into from
Jan 24, 2018
Merged

Add verbatim LICENSE #6

merged 1 commit into from
Jan 24, 2018

Conversation

alexdima
Copy link
Contributor

@alexdima alexdima commented Dec 7, 2017

Hi, I'm working on VSCode and we would like to attribute your node module in our ThirdPartyNotices.txt (yay! we're shipping it!)

We have a pretty basic automated generator that picks up LICENSE files from repositories and it is pretty standard that OSS projects have some form of license file at their root.

It would be great if we could remove our custom rule we currently have in place for your repository (as it is missing a license file) and simply pick the license straight from your repository, with the wording you would like.

This will also help other projects that want to consume your node module.

Thanks,
Alex

Hi, I'm working on VSCode and we would like to attribute your node module in our ThirdPartyNotices.txt (yay! we're shipping it!)

We have a pretty basic automated generator that picks up LICENSE files from repositories and it is pretty standard that OSS projects have some form of license file at their root.

It would be great if we could remove our custom rule we currently have in place for your repository (as it is missing a license file) and simply pick the license straight from your repository, with the wording you would like.

This will also help other projects that want to consume your node module.

Thanks,
Alex
@coveralls
Copy link

Coverage Status

Coverage remained the same at 99.712% when pulling 86874e4 on alexandrudima:patch-1 into 500f26f on nojvek:master.

Copy link
Owner

@nojvek nojvek left a comment

Choose a reason for hiding this comment

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

@nojvek nojvek merged commit d50156a into nojvek:master Jan 24, 2018
alexdima added a commit to microsoft/vscode that referenced this pull request Feb 1, 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
Development

Successfully merging this pull request may close these issues.

3 participants