-
-
Notifications
You must be signed in to change notification settings - Fork 154
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
feat(gitlab): Add gitlab support #612
Conversation
Thanks for opening this pull request! Please check out our contributing guidelines! ⛰️ |
Codecov ReportAttention: Patch coverage is
❗ Your organization needs to install the Codecov GitHub app to enable full functionality. Additional details and impacted files@@ Coverage Diff @@
## main #612 +/- ##
==========================================
- Coverage 41.52% 38.96% -2.56%
==========================================
Files 15 16 +1
Lines 1072 1258 +186
==========================================
+ Hits 445 490 +45
- Misses 627 768 +141
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
2d2f871
to
f715ef6
Compare
700783a
to
6d6f256
Compare
@orhun I have tried to stick as close as I can to your github integration. Fortunately the gitlab api works in a very similar way to the github one. Please tell me if you think this is the right approach. |
6755bff
to
cf36951
Compare
Wow, this is great! Thanks for making it happen <3 Is it actually functional now? Can we add some tests (fixtures) to verify the behavior? |
Yes I will add some tests in a bit! Glad to see you happy for it! |
644d052
to
7160c6d
Compare
@orhun I have a working integration test 🎉 Some slight bugs remain but we are almost there |
75ed80b
to
8d7f23c
Compare
@orhun this is now complete, with tests and integration tests. I would appreciate it if you could look over it. |
e1b2875
to
a41c6a1
Compare
@orhun let me know if you need anything more for me to merge this. Thank you! |
Thank you @dark0dave - this is pretty exciting. I will add a review soon! |
@orhun I have significantly reduced the fields in the gitlab merge request struct as requested. If you would kindly take another look, thank you! |
Signed-off-by: dark0dave <dark0dave@mykolab.com>
* feat(update): Remove vim by @dark0dave | ||
|
||
## New Contributors | ||
* @dark0dave made their first contribution in # |
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.
Any idea why the MR number is empty? Maybe we can avoid this by checking it in the template.
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.
Its because I am skipping merge commits.
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.
I'll add back in
Hey, all looks good! One thing that I want to do before moving on with this is refactoring the code to avoid duplications between GitHub and GitLab integration. This will also make it easier to add other remotes in the future. My plan is to introduce a trait (maybe called I can get to this this week, but let me know what you think of it and possibly want to take a stab! Sorry it took a bit of time to get this PR in - due to the fact that it was a great surprise 😄 |
Yeah I was thinking about that, the whole time I was doing this. It would make it easier for us to support more providers in the future. I was thinking we could do it as an improvement (ie annother PR). But happy to do it to this one if you think thats appropriate. |
Feel free to do it in this PR! I think I will also have some time this week, but feel free to beat me to it! |
Hey @dark0dave! I made the changes in another PR since I don't have access to push to this PR: #654 I think the only thing left is to update the documentation and then release! 🚀 |
@dark0dave thank you a lot once again! 💖 |
Description
Adding Gitlab support
Motivation and Context
Much like we support github we should also support gitlab
How Has This Been Tested?
Its still a work in progress
Screenshots / Logs (if applicable)
Types of Changes
Checklist: