Skip to content
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

Label support for pull requests #23

Merged
merged 11 commits into from
Jul 25, 2014
Merged

Label support for pull requests #23

merged 11 commits into from
Jul 25, 2014

Conversation

elliothursh
Copy link
Contributor

Rollback Plan

If this pull request requires anything more complex (e.g., rolling back a migration), you MUST update this section. Otherwise, delete this note.

To roll back this change, revert the merge with git revert -m 1 MERGE_SHA and perform another deploy.

URLs

QA Plan

Run op pull-request, make sure it asks you a question about labels. then check to make sure the label got made.

@@ -60,6 +60,10 @@ def title
pull_request_data.title
end

# def number

Choose a reason for hiding this comment

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

NOPE don't do this.

@@ -122,6 +133,11 @@ def update_jira
end
private :update_jira

def update_label
pull_request.add_labels(pull_request.repo_name, pull_request.number, label) unless (label == "Don't know yet")

Choose a reason for hiding this comment

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

you still have the number in here.

Choose a reason for hiding this comment

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

and the repo

@anfleene
Copy link

This looks good 👍

@elliothursh elliothursh changed the title Risk Labels Labels Jul 25, 2014
@NickLaMuro NickLaMuro changed the title Labels Label support for pull requests Jul 25, 2014
elliothursh added a commit that referenced this pull request Jul 25, 2014
Label support for pull requests
@elliothursh elliothursh merged commit 8ba1ba3 into master Jul 25, 2014
@emmahsax emmahsax deleted the risk-labels branch July 14, 2020 08:00
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.

None yet

2 participants