Skip to content

[PR #5098/aab22e7f backport][stable-5] Add remaining copyright statements, add licenses sanity test#5099

Merged
felixfontein merged 2 commits intostable-5from
patchback/backports/stable-5/aab22e7f323206ddc735d3c5157ca975c03cf771/pr-5098
Aug 14, 2022
Merged

[PR #5098/aab22e7f backport][stable-5] Add remaining copyright statements, add licenses sanity test#5099
felixfontein merged 2 commits intostable-5from
patchback/backports/stable-5/aab22e7f323206ddc735d3c5157ca975c03cf771/pr-5098

Conversation

@patchback
Copy link

@patchback patchback bot commented Aug 8, 2022

This is a backport of PR #5098 as merged into main (aab22e7).

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

* 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 changed the title [PR #5098/aab22e7f backport][stable-5] Add remaining copyright statements, add licenses sanity test [WIP] [PR #5098/aab22e7f backport][stable-5] Add remaining copyright statements, add licenses sanity test Aug 8, 2022
@ansibullbot
Copy link
Collaborator

@ansibullbot ansibullbot added WIP Work in progress backport docs integration tests/integration module module module_utils module_utils monitoring net_tools new_contributor Help guide this first time contributor plugins plugin (any type) tests tests unit tests/unit labels Aug 8, 2022
@codecov
Copy link

codecov bot commented Aug 9, 2022

Codecov Report

❗ No coverage uploaded for pull request base (stable-5@0d598c3). Click here to learn what that means.
The diff coverage is n/a.

❗ Current head aab22e7 differs from pull request most recent head 0b520e2. Consider uploading reports for the commit 0b520e2 to get more accurate results

@@             Coverage Diff             @@
##             stable-5    #5099   +/-   ##
===========================================
  Coverage            ?   45.46%           
===========================================
  Files               ?      974           
  Lines               ?    95980           
  Branches            ?    18273           
===========================================
  Hits                ?    43637           
  Misses              ?    50359           
  Partials            ?     1984           
Flag Coverage Δ
integration 71.51% <0.00%> (?)
sanity 21.36% <0.00%> (?)
stub 0.00% <0.00%> (?)
units 68.26% <0.00%> (?)

Flags with carried forward coverage won't be shown. Click here to find out more.

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@felixfontein
Copy link
Collaborator

Once #5106 is merged I will cherry-pick its commit into here as well.

(cherry picked from commit c892505)
@felixfontein felixfontein changed the title [WIP] [PR #5098/aab22e7f backport][stable-5] Add remaining copyright statements, add licenses sanity test [PR #5098/aab22e7f backport][stable-5] Add remaining copyright statements, add licenses sanity test Aug 13, 2022
@ansibullbot ansibullbot removed the WIP Work in progress label Aug 13, 2022
@felixfontein felixfontein merged commit bc24030 into stable-5 Aug 14, 2022
@felixfontein felixfontein deleted the patchback/backports/stable-5/aab22e7f323206ddc735d3c5157ca975c03cf771/pr-5098 branch August 14, 2022 08:26
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 new_contributor Help guide this first time contributor plugins plugin (any type) tests tests unit tests/unit

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants