smccarthy788 opened a new pull request, #37: URL: https://github.com/apache/otava/pull/37
Closes #30. Closes #31. The main criteria I used to choose how to automate the license header enforcement was integrating into our existing CI built around `poetryw` & `toxw`. In particular I didn't want to introduce downloading some binary or using another language to run a tool. Turns out that was a lot to ask for... I didn't find any good options until I came across how this is done in [Apache Airflow](https://github.com/apache/airflow), using `pre-commit`. Still... IMO commit hooks are kind of clunky, I'm in no way wedded to this approach. At the very least: the license headers have been added. The license headers can be applied to new files by running the normal formatting command: ``` ./toxw -e format ``` ... and being part of the tox testenv will also be tested in CI. There are some extra steps to update the license header, documented in `CONTRIBUTING.md`. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
