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 License Field to package.json for Clarity and Compatibility #89

Merged
merged 1 commit into from
Nov 19, 2023

Conversation

10xLaCroixDrinker
Copy link
Contributor

I've noticed that while you have a LICENSE file, the license isn't specified in the package.json. I'd like to propose a small yet significant enhancement by adding the license information to your package.json. This change offers several benefits:

  • Immediate Visibility: It provides instant clarity regarding the licensing terms right from the package metadata, which is beneficial for users and developers exploring your package.
  • Tool Compatibility: Automated tools and services, like npm and GitHub, rely on the package.json file to detect and display license information. This enhancement ensures your project is correctly recognized in various ecosystems.
  • Community Best Practices: Including the license in package.json is a common practice in the open-source community. It helps in maintaining consistency with these standards.
  • Legal Clarity: Specifying the license in both the package.json and the LICENSE file eliminates ambiguity about your project's licensing terms, which is crucial for legal clarity.

@CLAassistant
Copy link

CLAassistant commented Nov 15, 2023

CLA assistant check
All committers have signed the CLA.

@guybedford
Copy link
Member

Sure I'd be happy to add this - note that the next release may be up to a couple of months out yet though as well.

@guybedford guybedford merged commit 7af7d74 into jspm:main Nov 19, 2023
1 of 3 checks passed
@10xLaCroixDrinker 10xLaCroixDrinker deleted the chore/license branch April 27, 2024 06:20
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.

None yet

3 participants