-
Notifications
You must be signed in to change notification settings - Fork 2
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
Testing: git hooks #8
Comments
use git hooks to:
|
I think that we have implemented this using Github actions and this should now be closed. Do you agree @zmaalick |
Git hooks are different. They lie in local repo. You can see under ~/CATNIP/.git/hooks the concept is similar to Actions but at the time of commit or push .. so these are mainly pre-commit or pre-push checks (not necessarily include all unit tests). |
I understand the difference. What I am asking is, are there still things we want to implement using git hooks? |
Not sure, I do not have any specific test in mind to add here .
From: Nick Savage <notifications@github.com>
Sent: 21 September 2020 14:01
To: MetOffice/CATNIP <CATNIP@noreply.github.com>
Cc: Maalick, Zubair <zubair.maalick@metoffice.gov.uk>; Mention <mention@noreply.github.com>
Subject: Re: [MetOffice/CATNIP] Testing: git hooks (#8)
I understand the difference. What I am asking is, are there still things we want to implement using git hooks?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub<#8 (comment)>, or unsubscribe<https://github.com/notifications/unsubscribe-auth/AN2CPNARXOP2O5F5GPLLINDSG5E7ZANCNFSM4K5M6VEQ>.
|
I will close this - if we find a use case for hooks. we should open a new issue with the details |
Make use of git hooks
The text was updated successfully, but these errors were encountered: