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

Make reuse conformant #5138

Merged
merged 8 commits into from
Aug 21, 2022
Merged

Conversation

felixfontein
Copy link
Collaborator

SUMMARY

Everything except changelog/fragments/ is now reuse conformant (https://reuse.software/spec/).

CC @gotmax23

ISSUE TYPE
  • Docs Pull Request
  • Feature Pull Request
COMPONENT NAME

licensing

@ansibullbot ansibullbot added the needs_revision This PR fails CI tests or a maintainer has requested a review/revision of the PR label Aug 20, 2022
@gotmax23
Copy link
Contributor

Everything except changelog/fragments/

If you wanted, you could add a .reuse/dep5 file to the repo root with the following contents:

Format: https://www.debian.org/doc/packaging-manuals/copyright-format/1.0/

Files: changelogs/fragments/*
Copyright: Ansible Project
License: GPL-3.0-or-later

Then, the entire repository would be fully REUSE compliant1 and it could get a REUSE compliant badge.

Footnotes

  1. without having to add headers that take up more room than the actual changelog fragment

@felixfontein
Copy link
Collaborator Author

@gotmax23 great idea, I've added that file.

@ansibullbot ansibullbot removed the needs_revision This PR fails CI tests or a maintainer has requested a review/revision of the PR label Aug 20, 2022
README.md Outdated Show resolved Hide resolved
@felixfontein felixfontein merged commit 02d9a1d into ansible-collections:main Aug 21, 2022
@felixfontein felixfontein deleted the licenses-2 branch August 21, 2022 06:27
@felixfontein
Copy link
Collaborator Author

@gotmax23 thanks for reviewing this!

@patchback
Copy link

patchback bot commented Aug 21, 2022

Backport to stable-5: 💚 backport PR created

✅ Backport PR branch: patchback/backports/stable-5/02d9a1de9e840664e6516f6c11c82488cadda090/pr-5138

Backported as #5139

🤖 @patchback
I'm built with octomachinery and
my source is open — https://github.com/sanitizers/patchback-github-app.

patchback bot pushed a commit that referenced this pull request Aug 21, 2022
* Add .license files.

* Add reuse test.

* Update README.

* Also remove .yaml changelog fragments.

* Add changelog fragment.

* Normalize licenses extra sanity test.

* Declare REUSE conformance.

* Update README.md

(cherry picked from commit 02d9a1d)
felixfontein added a commit that referenced this pull request Aug 21, 2022
* Add .license files.

* Add reuse test.

* Update README.

* Also remove .yaml changelog fragments.

* Add changelog fragment.

* Normalize licenses extra sanity test.

* Declare REUSE conformance.

* Update README.md

(cherry picked from commit 02d9a1d)

Co-authored-by: Felix Fontein <felix@fontein.de>
@mxmehl
Copy link

mxmehl commented Oct 6, 2022

Great job, thanks for making this effort!

What do you think about getting a badge for this? The REUSE projects offers registering a repo with their API: https://api.reuse.software

bratwurzt pushed a commit to bratwurzt/community.general that referenced this pull request Nov 7, 2022
* Add .license files.

* Add reuse test.

* Update README.

* Also remove .yaml changelog fragments.

* Add changelog fragment.

* Normalize licenses extra sanity test.

* Declare REUSE conformance.

* Update README.md
bratwurzt pushed a commit to bratwurzt/community.general that referenced this pull request Nov 7, 2022
* Add .license files.

* Add reuse test.

* Update README.

* Also remove .yaml changelog fragments.

* Add changelog fragment.

* Normalize licenses extra sanity test.

* Declare REUSE conformance.

* Update README.md
This was referenced Nov 20, 2022
@mxmehl mxmehl mentioned this pull request May 14, 2024
1 task
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants