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

copyright and encoding pragma pre-commit hooks #13

Merged
merged 1 commit into from
Apr 16, 2024

Conversation

mattwwarren
Copy link
Collaborator

No description provided.

@mattwwarren mattwwarren merged commit ba791f3 into main Apr 16, 2024
1 check passed
@mattwwarren mattwwarren deleted the add-copyright-pre-commit branch April 16, 2024 21:21
@@ -0,0 +1,4 @@
#
# Copyright (C) CloudTruth, Inc.

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is missing a year, which I think we want to have https://termly.io/resources/articles/copyright-examples/

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Initially I was thinking we couldn't have a year because nothing would update it. Then I remembered humans.

- repo: https://github.com/leoll2/copyright_notice_precommit
rev: 0.1.1
hooks:
- id: copyright-notice

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Do we want to drop this into conserv too?

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Totally can. I was just testing here to see how it works.

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

Successfully merging this pull request may close these issues.

2 participants