Skip to content

Conversation

Ebolon
Copy link
Contributor

@Ebolon Ebolon commented Jul 19, 2013

Current API uses a string to indicate "closed" and "merged" state. In future we have two boolean variables [1]. For backward reasons we parse the state in the model.

[1] https://github.com/gitlabhq/gitlabhq/blob/master/doc/api/merge_requests.md

…d state parse to GitlabMergeRequest model.
@Ebolon
Copy link
Contributor Author

Ebolon commented Jul 19, 2013

Oh it should be "closed" or "merged" :-)

timols added a commit that referenced this pull request Jul 22, 2013
Fixed to check only merge request with state "closed" or "open".
@timols timols merged commit 3b42ed3 into timols:master Jul 22, 2013
timols pushed a commit that referenced this pull request Dec 15, 2015
update from timols/java-gitlab-api
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants