-
Notifications
You must be signed in to change notification settings - Fork 51
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
GitLab API support #26
Comments
+1 |
1 similar comment
+1 |
You know there are plenty of Gitlab clients to choose from: https://about.gitlab.com/applications/ |
+1 |
2 similar comments
👍 |
👍 |
Has anyone tried this yet? Just looking at it, and thinking about giving it a try |
+1 |
For anyone that is interested in claiming the GitLab support bounty. Feel free to contact me personally via email or DM me on twitter if you have any questions regarding the somewhat unusual metaprogramming techniques in Ghee or my opinions on how the GitLab integration should best be implemented Contant info: Ryan Rauh |
I'm curious how you envision managing the differences between GitLab and GitHub APIs. For example, GitHub has a concept of organizations and teams, while GitLab only has namespaces/groups. Also, naming is a bit different:
I have a feeling that some work would have to be made in the GitLab API to include all the features used by ghee. For example, the filtering parameters may not all be supported (e.g. |
I think the easiest way is to make Ghee behave as similarly to the GitHub API as possible. Perhaps Ghee can be an abstraction that provides the lowest common denominator to both API's
HuBoard currently doesn't use this functionality, it's probably "ok" to have some filters that silently don't work when in GitLab mode. I think the first step to GitLab support, or at least where |
It's worth noting that whomever picks up this bounty can expect that myself and my team will be available to mentor and help whenever needed |
Thanks, @rauhryan. I'll take a crack at it since I'm fairly familiar with the GitLab and GitHub APIs. |
Awesome! Open a PR as early as possible and we'll keep an eye on it and give you feedback. Let me know if you need help getting the test suite going, it's currently a bit of a pain in the ass because it's technically integration tests directly against the GitHub API |
@rauhryan @stanhu , I propose we divide individual steps for this project into Ghee issues with the label "GitLab", and track them via https://huboard.com/huboard/ghee |
@rauhryan Not sure if this is the right place to ask, but I've set up my
I've verified the connection is setting the headers appropriately. Have you seen this before? |
That test is particularly interesting, it requires a username and password and basic auth. Please ignore it for the time being because I wouldn't recommend putting your username and password on disk in the settings.yml Is there a way to tag a test as explict in rspec? |
See specification here:
https://gitlab.com/gitlab-org/gitlab-ce/tree/master/doc/api
The text was updated successfully, but these errors were encountered: