Skip to content
This repository has been archived by the owner on Jul 12, 2022. It is now read-only.

Support labels for GitLab merge requests #832

Merged
merged 2 commits into from
Jul 1, 2019

Conversation

zubivan
Copy link
Contributor

@zubivan zubivan commented Jun 27, 2019

✨ What kind of change does this PR introduce? (Bug fix, feature, docs update...)

Brings support for label to GitLab merge requests.

⤵️ What is the current behavior?

We don't support it.

🆕 What is the new behavior (if this is a feature change)?

We support it.

💥 Does this PR introduce a breaking change?

Nope.

🐛 Recommendations for testing

Try to create a gitlab MR.

📝 Links to relevant issues/docs

Already supported by the API. https://docs.gitlab.com/ee/api/merge_requests.html#create-mr
Just never bother to implement it.

🤔 Checklist before submitting

  • All projects build
  • Relevant documentation was updated

@@ -21,5 +21,8 @@ public class MergeRequest

[JsonProperty("remove_source_branch")]
public bool RemoveSourceBranch { get; set; }

[JsonProperty("lables")]
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think this should be labels.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Haha, damn, I was quite sure to copy it. Silly, thanks for taking a look, Max!

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No problem, that is why we do code reviews :)

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

And that's fixed... 🤦‍♂

@MaxMommersteeg
Copy link
Contributor

Small typo, but LGTM once fixed.

@MarcBruins
Copy link
Member

Thanks Ivan!

@MarcBruins MarcBruins merged commit fe974fa into NuKeeperDotNet:master Jul 1, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants