Skip to content

Fixed to check only merge request with state "closed" or "open". #1

Merged
timols merged 1 commit intotimols:masterfrom
Ebolon:merge-state
Jul 22, 2013
Merged

Fixed to check only merge request with state "closed" or "open". #1
timols merged 1 commit intotimols:masterfrom
Ebolon:merge-state

Conversation

@Ebolon
Copy link
Copy Markdown
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
Copy Markdown
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