Skip to content

[perceval] Update license and copyright information#624

Merged
valeriocos merged 1 commit intochaoss:masterfrom
vchrombie:test-copyright-script
Mar 13, 2020
Merged

[perceval] Update license and copyright information#624
valeriocos merged 1 commit intochaoss:masterfrom
vchrombie:test-copyright-script

Conversation

@vchrombie
Copy link
Copy Markdown
Member

@vchrombie vchrombie commented Mar 8, 2020

This PR updates license information for the remaining source code files in Perceval.
This is an extension of #623.

I tried updating this using the tool, vchrombie/grimoirelab-scripts.

Comment thread tests/test_pipermail.py
@vchrombie vchrombie force-pushed the test-copyright-script branch 2 times, most recently from e34cbf0 to 7f0d599 Compare March 8, 2020 20:22
@valeriocos
Copy link
Copy Markdown
Member

valeriocos commented Mar 8, 2020

can you update the commit description by adding some info about the files modified (e.g., tests, bin and setup.py)? thanks. A possible description could be:

This commit updates the license information for every test file in Perceval, the setup.py and bin/perceval. The author names are also updated using a script.

Please, consider to submit also PRs to ELK and mordred, and the other components :)

@vchrombie
Copy link
Copy Markdown
Member Author

Please, consider to submit also PRs to ELK and mordred, and the other components :)

Sure @valeriocos.

@vchrombie vchrombie force-pushed the test-copyright-script branch from 7f0d599 to 186b179 Compare March 9, 2020 04:59
@vchrombie vchrombie marked this pull request as ready for review March 11, 2020 01:20
@vchrombie
Copy link
Copy Markdown
Member Author

I just had one doubt in this PR. #624 (comment)

The rest all looks good to me.

@valeriocos, WDYT?

@valeriocos
Copy link
Copy Markdown
Member

Please @vchrombie ping me when the PR is ready for review, thanks!

@vchrombie vchrombie force-pushed the test-copyright-script branch from 186b179 to 0701201 Compare March 12, 2020 20:18
This commit updates the license information for every
test file in Perceval, the setup.py and bin/perceval.
The author names are also updated using a script.

Signed-off-by: Venu Vardhan Reddy Tekula <[email protected]>
@vchrombie vchrombie force-pushed the test-copyright-script branch from 0701201 to 07b0998 Compare March 12, 2020 20:41
@coveralls
Copy link
Copy Markdown

coveralls commented Mar 12, 2020

Coverage Status

Coverage remained the same at 97.663% when pulling 07b0998 on vchrombie:test-copyright-script into 75be46b on chaoss:master.

@vchrombie
Copy link
Copy Markdown
Member Author

Hi @valeriocos

I updated the PR and I think it is in good shape now. The Travis CI build also passed. 😌
You can review it now.

@vchrombie vchrombie requested a review from valeriocos March 12, 2020 20:55
Copy link
Copy Markdown
Member

@valeriocos valeriocos left a comment

Choose a reason for hiding this comment

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

LGTM, thanks @vchrombie

@valeriocos valeriocos merged commit 807e640 into chaoss:master Mar 13, 2020
@vchrombie vchrombie deleted the test-copyright-script branch November 3, 2020 09:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants