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 remaining copyright statements, add licenses sanity test #5098

Merged
merged 8 commits into from
Aug 8, 2022

Conversation

felixfontein
Copy link
Collaborator

SUMMARY

This should conclude the endeavor for community.general :-)

CC @gotmax23

CC @russoz and @jirihnidek for eb36302

The statements I'm most unsure of are 7ca7fb0

ISSUE TYPE
  • Docs Pull Request
COMPONENT NAME

various

@ansibullbot
Copy link
Collaborator

@ansibullbot ansibullbot added docs integration tests/integration module module module_utils module_utils monitoring net_tools plugins plugin (any type) tests tests unit tests/unit labels Aug 8, 2022
@felixfontein felixfontein merged commit aab22e7 into ansible-collections:main Aug 8, 2022
@patchback
Copy link

patchback bot commented Aug 8, 2022

Backport to stable-5: 💚 backport PR created

✅ Backport PR branch: patchback/backports/stable-5/aab22e7f323206ddc735d3c5157ca975c03cf771/pr-5098

Backported as #5099

🤖 @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 8, 2022
* Add sanity test (currently fails).

* doc_fragments can also be non-GPLv3+.

* Replace 'Author:' by 'Copyright:' in some specific cases.

* Avoid matching string for license checkers.

* Reformulate not to throw license detection off.

* Add PSF copyright notice for plugins/module_utils/_mount.py.

* Add generic copyright notices.

* Update changelog fragment.

(cherry picked from commit aab22e7)
@felixfontein felixfontein deleted the licenses branch August 8, 2022 21:23
@felixfontein
Copy link
Collaborator Author

Ok, that was a bit too early... I merged the wrong tab. Sorry for that! I won't merge the backport until this - and possibly follow-up PRs to fix what's wrong here - are done.

Copy link
Contributor

@gotmax23 gotmax23 left a comment

Choose a reason for hiding this comment

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

I have a couple nitpicks, but I don't think it's a big deal that you merged it as is.

@felixfontein
Copy link
Collaborator Author

I'm collecting updates to this PR in #5106.

felixfontein added a commit that referenced this pull request Aug 14, 2022
…ents, add licenses sanity test (#5099)

* Add remaining copyright statements, add licenses sanity test (#5098)

* Add sanity test (currently fails).

* doc_fragments can also be non-GPLv3+.

* Replace 'Author:' by 'Copyright:' in some specific cases.

* Avoid matching string for license checkers.

* Reformulate not to throw license detection off.

* Add PSF copyright notice for plugins/module_utils/_mount.py.

* Add generic copyright notices.

* Update changelog fragment.

(cherry picked from commit aab22e7)

* Update copyright statements. (#5106)

(cherry picked from commit c892505)

Co-authored-by: Felix Fontein <[email protected]>
bratwurzt pushed a commit to bratwurzt/community.general that referenced this pull request Nov 7, 2022
…-collections#5098)

* Add sanity test (currently fails).

* doc_fragments can also be non-GPLv3+.

* Replace 'Author:' by 'Copyright:' in some specific cases.

* Avoid matching string for license checkers.

* Reformulate not to throw license detection off.

* Add PSF copyright notice for plugins/module_utils/_mount.py.

* Add generic copyright notices.

* Update changelog fragment.
This was referenced Nov 20, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
docs integration tests/integration module_utils module_utils module module monitoring net_tools plugins plugin (any type) tests tests unit tests/unit
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants