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 headers check #7

Open
rebstar6 opened this issue Oct 19, 2018 · 2 comments
Open

Add headers check #7

rebstar6 opened this issue Oct 19, 2018 · 2 comments
Labels
good first issue Good for newcomers help wanted Extra attention is needed

Comments

@rebstar6
Copy link
Contributor

In this repo, there are few enough files (and likely wont be many more) that I think a pre-commit hook should work.

If there is some built-in python way to do this, do that. If not, a script is fine. See https://github.com/vinyldns/vinyldns/blob/master/bin/add-license-headers.sh for an example of how we scripted this for the non-scala files in the api

@Chalmiller
Copy link

I'm looking to start contributing to vinyldns, so I'll take on this task if it's still valid?

@remerle
Copy link
Member

remerle commented Aug 26, 2021

@Chalmiller we welcome any contributions! I'd eventually like to transition this project to using GitHub Actions since the main VinylDNS project is now using them. So this would be a good issue to start that train moving 🙂

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
good first issue Good for newcomers help wanted Extra attention is needed
Projects
None yet
Development

No branches or pull requests

6 participants