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

FEATURE: Added license check header pre commit hook support #438

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

drewm102
Copy link
Collaborator

closes #351

@drewm102 drewm102 changed the title F351: Added license check header pre commit hook support FEATURE: Added license check header pre commit hook support Oct 11, 2024
@chenchienjacklin
Copy link
Collaborator

chenchienjacklin commented Oct 16, 2024

@drewm102 All these look good. I think we can have another issue for setting up a GitHub Action to update year(s) in each file. This is an example of updating the license file. https://github.com/marketplace/actions/update-license-copyright-year-s
Do you think it's a good idea to avoid manual update?

@drewm102
Copy link
Collaborator Author

@drewm102 All these look good. I think we can have another issue for setting up a GitHub Action to update year(s) in each file. This is an example of updating the license file. https://github.com/marketplace/actions/update-license-copyright-year-s Do you think it's a good idea to avoid manual update?

@chenchienjacklin I think this is a great idea! I will look into adding these changes to this PR as well

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.

Integrate pyansys precommit hook for license statements
2 participants