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 Apache-2.0 license #89

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

paulbrimicombe
Copy link
Contributor

What?

  • Add a LICENSE file with the Apache 2.0 license (the license we usually use at the BBC for open source projects).
  • Add licenses to npm package.json files
  • Set npm packages to be private to avoid them being accidentally published to the NPM registry (these projects are not really for distribution like that)

Why?

  • This repository is open source so we should be explicit about the license that we are expecting users to abide by

Copy link
Member

@nicholasgriffintn nicholasgriffintn left a comment

Choose a reason for hiding this comment

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

Makes sense to me, might be something for review though.

@@ -8,7 +8,8 @@
"test": "jest"
},
"author": "BBC News and Weather Apps",
"license": "ISC",
"private": true,
Copy link
Member

Choose a reason for hiding this comment

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

Don't think we need this line?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

This there to stop people accidentally publishing the project to NPM (because that's not really something we want to do with this project — it's not intended to be installed via a registry, it's intended to be checked-out from GitHub).

@CameronWatt98
Copy link

This all makes sense to me.

Copy link
Contributor

@TimPerry TimPerry left a comment

Choose a reason for hiding this comment

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

Yup yup, makes sense. LGTM.

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.

4 participants