-
Notifications
You must be signed in to change notification settings - Fork 15
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 a Contributor Agreement #42
Conversation
…mit hashes, add security hardening
for more information, see https://pre-commit.ci
for more information, see https://pre-commit.ci
…warning to new pull requests about agreeing to the license
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
looks good. Sorry for the late review! Only need to change version number or remove it so this doesn't have to be regularly updated.
So I don't break anything with my novice Git status- do you want to update the branch to merge with the main, and I'll merge the pull request afterwards? |
@analytophile Yes, that's exactly how I'll go forward. Once a Pull Request is "approved", anyone with write access here can merge a Pull Request, so I'll make the remaining changes and merge when it's ready. Thanks again! |
Pull Request Checklist:
What kind of change does this PR introduce?
Does this PR introduce a breaking change?
Other information: