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

adding functionality to set labels #70

Merged
merged 6 commits into from
May 3, 2018
Merged

Conversation

dr-rodriguez
Copy link
Contributor

Some additional methods to IssueHandler to allow for labels to be set (#54). Also added a line in stale issues to add the 'closed-by-bot' label on issues that are auto-closed.

Also relevant to issues #65 and #66

Tests are still needed for the new method.

@coveralls
Copy link

coveralls commented May 3, 2018

Coverage Status

Coverage increased (+0.3%) to 85.147% when pulling b01f4ab on dr-rodriguez:set-label into 697a1ea on astropy:master.

@pllim
Copy link
Member

pllim commented May 3, 2018

@bsipocz do you want to have a look before I merge? @astrofrog are you still awake? If I don't hear by 6PM, I'll just merge as is. This LGTM. Thanks!

@bsipocz
Copy link
Member

bsipocz commented May 3, 2018

@pllim - Let's merge :)

@bsipocz bsipocz merged commit 872857b into astropy:master May 3, 2018
@bsipocz
Copy link
Member

bsipocz commented May 3, 2018

Thanks @dr-rodriguez and @pllim!

@pllim
Copy link
Member

pllim commented May 3, 2018

Huh... I can see the "close and comment" button but I cannot see "merge".

@pllim
Copy link
Member

pllim commented May 3, 2018

Oh, it is merged already. Thanks, @bsipocz . Hahaha...

@dr-rodriguez dr-rodriguez deleted the set-label branch May 4, 2018 15:06
astrofrog pushed a commit to OpenAstronomy/baldrick that referenced this pull request Jul 19, 2018
astrofrog pushed a commit to OpenAstronomy/baldrick that referenced this pull request Jul 19, 2018
astrofrog pushed a commit to OpenAstronomy/baldrick that referenced this pull request Jul 23, 2018
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

4 participants