HorizenOfficial / zen

Horizen
https://www.horizen.io
Other
264 stars 120 forks source link

Update copyright headers #644

Closed ptagl closed 5 months ago

ptagl commented 6 months ago

This pull request introduces the following changes:

The previous script fix-copyright-headers.py has been removed and the devtools readme has been updated accordingly.

ptagl commented 5 months ago

@drgora I've added a basic mechanism to exclude commits that we shouldn't take into account when updating copyright headers (like the updates of the copyright headers themselves)! 😄

The mechanism is based on a new --exclude-commit argument (list) with the following behavior:

Let me know what you think about it.

Thanks!

drgora commented 5 months ago

@drgora I've added a basic mechanism to exclude commits that we shouldn't take into account when updating copyright headers (like the updates of the copyright headers themselves)! 😄

The mechanism is based on a new --exclude-commit argument (list) with the following behavior:

* if the argument is **empty** (not provided by the user), the script tries to automatically exclude commits containing `update-copyright-headers.py` inside the message or `copyright` inside the summary (I didn't want to check the whole message here just to prevent accidental exclusion of relevant commits); after the automatic detection, user is asked to confirm that the list of commits is fine, with the possibility to continue, abort or ignore the list

* if the list is **provided** by the user, no automatic detection is performed

Let me know what you think about it.

Thanks!

Sounds great to me, thanks!