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 configured license amendment content #624

Merged
merged 3 commits into from
Feb 8, 2023
Merged

Conversation

jonabc
Copy link
Contributor

@jonabc jonabc commented Feb 5, 2023

This change allows users to specify file paths to custom license terms provided by a library owner. Additional terms content files paths are specified per dependency type and name, and the files are treated as additional licenses for the dependency. See the added documentation for details on how to use the added additional_terms configuration option.

/cc @mlinksva I'm curious for your thoughts here - is this an appropriate change?

  1. It seems like there is some amount of risk in allowing users to specify content that would be treated as legally binding
  2. Should the extra terms be treated as a legal notice rather than a license?

UPDATE renamed amendments to "additional terms"

@jonabc
Copy link
Contributor Author

jonabc commented Feb 8, 2023

Ignoring the cabal failures, those tests are having issues during their setup steps. I'm merging this in and will put out a release once the cabal tests are fixed in a separate PR.

@jonabc jonabc merged commit ee0355d into master Feb 8, 2023
@jonabc jonabc deleted the license-amendments branch February 8, 2023 16:34
jonabc added a commit that referenced this pull request Feb 8, 2023
### Added

- Custom license terms can be added to dependencies via new configuration options (#624)
- Licensed is now integrated with pnpm to enumerate dependencies (#626)
@jonabc jonabc mentioned this pull request Feb 8, 2023
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.

1 participant