Skip to content

Conversation

@silvanocerza
Copy link
Contributor

No description provided.

Copy link
Contributor

@kittaakos kittaakos left a comment

Choose a reason for hiding this comment

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

Please check other occurrences too. Thanks!

@silvanocerza silvanocerza force-pushed the scerza/package-license branch from 6564c4f to 8facd20 Compare February 26, 2021 13:45
@silvanocerza
Copy link
Contributor Author

Please check other occurrences too. Thanks!

Updated. 👍

Copy link
Contributor

@per1234 per1234 left a comment

Choose a reason for hiding this comment

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

I confirmed this is using the valid SPDX identifier:
https://spdx.org/licenses/AGPL-3.0-or-later.html
as required by the package.json specification:
https://docs.npmjs.com/cli/v7/configuring-npm/package-json#license

I don't know enough to say whether the new license value is appropriate for the component associated with each of the files though.

Copy link
Contributor

@kittaakos kittaakos left a comment

Choose a reason for hiding this comment

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

Thank you!

@silvanocerza
Copy link
Contributor Author

Merging.

@silvanocerza silvanocerza merged commit 19aa3dd into main Feb 26, 2021
@silvanocerza silvanocerza deleted the scerza/package-license branch February 26, 2021 13:54
@per1234 per1234 added the topic: infrastructure Related to project infrastructure label Oct 26, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

topic: infrastructure Related to project infrastructure

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants