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

Use SPDX license identifiers for license #641

Open
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

leoheitmannruiz
Copy link

I'd love to see #221 implemented. This is my go at it!

@leoheitmannruiz
Copy link
Author

Oh, and, out of curiosity: in what context is license_url used?

@leoheitmannruiz
Copy link
Author

leoheitmannruiz commented Sep 12, 2024

I suppose, being super proper would entail accommodating SPDX license expressions.

crystal, for instance, is licensed Apache-2.0 WITH Swift-exception.

docs/shard.yml.adoc Outdated Show resolved Hide resolved
@leoheitmannruiz
Copy link
Author

leoheitmannruiz commented Sep 24, 2024

Thanks for the feedback!

What do you think about allowing for SPDX license expressions?

Edit: It seems to me that without them, it may not be possible to describe the licensing of a shard accurately. Such as in the case of crystal.

@straight-shoota
Copy link
Member

Expressions sounds good 👍
We don't need to do any specific processing for this, though. This is primarily an informative field anyway.

@leoheitmannruiz
Copy link
Author

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants