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 license headers to all files #13

Closed
acs opened this issue Feb 22, 2018 · 6 comments · Fixed by #199
Closed

Add license headers to all files #13

acs opened this issue Feb 22, 2018 · 6 comments · Fixed by #199

Comments

@acs
Copy link
Contributor

acs commented Feb 22, 2018

The GPLv3 header is not included in python souce files in the project. They need to be added.

@acs acs added this to the v0.4 milestone Mar 18, 2018
@acs acs modified the milestones: v0.4 , v0.5 Oct 3, 2018
@vchrombie
Copy link
Contributor

Hi @acs

I would like to help with this. Can I work on this ?

@valeriocos
Copy link
Contributor

sure! thank you @vchrombie . As a reference you can rely on the licenses of other components (https://github.com/chaoss/grimoirelab-perceval/blob/master/perceval/backends/core/github.py#L3).

@vchrombie
Copy link
Contributor

vchrombie commented Feb 27, 2020

Thanks @valeriocos

I will send a PR, to discuss any more things to add. Also on another note, the copyright information needs to be updated the perceval backend source files too. I meant the year.

@valeriocos
Copy link
Contributor

the copyright and author info should be probably updated for all components. Maybe this task could be automated with some scripts. If you want, we can discuss it in a separated issue, WDYT?

@vchrombie
Copy link
Contributor

the copyright and author info should be probably updated for all components. Maybe this task could be automated with some scripts. If you want, we can discuss it in a separated issue, WDYT?

Okay, I am up for it. Can I open an issue regarding this in the grimoirelab repository as it accounts for all the components?

@valeriocos
Copy link
Contributor

valeriocos commented Feb 27, 2020

sure! thanks

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants