-
Notifications
You must be signed in to change notification settings - Fork 9
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
Create the gitlab hooks via API #10
Comments
I have started work on this issue. Currently @ant31 do you have any suggestions on a low-impact way to test changes in this workflow? Or should creating those tests be seen as a prerequisite? /cc @joejulian |
project doesnt have e2e nor unit tests. Unless you have strong opinion about it, I am good with manual QA |
IMHO I would love to see tests. Any tests even if they're only a small fraction of what should be tested. If you're willing to add tests for your new code, please do. If there are things you cannot test, skip that thing and create an issue. Don't make writing tests a burden. |
@samba do you have a WIP for this issue ? |
@ant31 Nothing yet material, mostly exploratory, identifying which methods were no longer relevant if the If this is useful, I can share, however I believe your familiarity with the codebase will make you more effective. |
https://docs.gitlab.com/ce/api/projects.html#add-project-hook
The text was updated successfully, but these errors were encountered: