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

Backfill software licenses meta #876

Merged
merged 6 commits into from
Oct 22, 2021

Conversation

klkeys
Copy link
Contributor

@klkeys klkeys commented Oct 21, 2021

PR checklist

Closes #873 (for now?)

  • This comment contains a description of changes (with reason).

backfills licenses and harmonizes some license names. only applies to modules with meta.yml files at time of PR.

harmonized names are for common licenses only (e.g. MIT, BSD variants, Apache, MPL, GNU variants). no consensus (yet) for nonstandard licenses.

@klkeys klkeys self-assigned this Oct 21, 2021
@klkeys klkeys force-pushed the backfill_software_licenses_meta branch from b8a398a to be1382e Compare October 21, 2021 00:50
@klkeys klkeys marked this pull request as ready for review October 22, 2021 18:13
@klkeys klkeys force-pushed the backfill_software_licenses_meta branch from e20047b to a3a1ffb Compare October 22, 2021 18:36
@klkeys klkeys requested a review from a team October 22, 2021 18:42
`bcftools` is dual-licensed, use associative array to allow for multiple licenses

Co-authored-by: Michael L Heuer <[email protected]>
@klkeys klkeys merged commit 3aacd46 into nf-core:master Oct 22, 2021
@klkeys klkeys deleted the backfill_software_licenses_meta branch October 22, 2021 22:40
@heuermh
Copy link
Contributor

heuermh commented Oct 22, 2021

Thank you, @klkeys @rpetit3

@klkeys klkeys mentioned this pull request Oct 22, 2021
1 task
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.

update modules to include software licenses, if possible
3 participants