Skip to content

Conversation

@arturf
Copy link

@arturf arturf commented Apr 23, 2014

Useful for Gitlab v.6.8.1

@arturf
Copy link
Author

arturf commented Jun 24, 2014

@m4tthumphrey, please, give any feedback about this pull request :)

@alxshr
Copy link

alxshr commented Jul 28, 2014

Looks great! Exactly what I am missing in the current version

@jubianchi
Copy link
Contributor

@alxshr @arturf I'll try to review this ASAP and hopefully merge it ;)

@m4tthumphrey
Copy link
Contributor

Hey @arturf! Could you squash these commits and I'll merge? :)

@arturf
Copy link
Author

arturf commented Oct 2, 2014

@m4tthumphrey into one?

@m4tthumphrey
Copy link
Contributor

Yeh into one commit which describes the PR :)

@arturf
Copy link
Author

arturf commented Oct 2, 2014

ready

- getting MR in different states
- `labels` property for MR model
Conflicts:
	lib/Gitlab/Api/MergeRequests.php
	lib/Gitlab/Model/MergeRequest.php
	lib/Gitlab/Model/Project.php
m4tthumphrey added a commit that referenced this pull request Oct 2, 2014
Update for merge request api
@m4tthumphrey m4tthumphrey merged commit ad9a681 into GitLabPHP:master Oct 2, 2014
@jubianchi
Copy link
Contributor

@m4tthumphrey sorry I'm not helping much here but I'm working hard on the Gitlab side to make the API better. Once I finish my work there I'll come back here to help on PR and patches.

BTW, thanks for the work @everyone ;)

@m4tthumphrey
Copy link
Contributor

No worries at all @jubianchi, thank you for your work! We're all busy!! :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

4 participants