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

Pull-requests support or issues-only? #19

Open
kdeldycke opened this issue Jun 17, 2020 · 6 comments
Open

Pull-requests support or issues-only? #19

kdeldycke opened this issue Jun 17, 2020 · 6 comments
Assignees
Labels
damccorms problem now enhancement New feature or request good first issue Good for newcomers help wanted Extra attention is needed plz help

Comments

@kdeldycke
Copy link

Required Information

Entering this information will route you directly to the right team and expedite traction.

Question, Bug, or Feature?
Type: Question

Should this get assigned to @damccorm?
Assign to damccorm: yes

Issue Description

Does that plugin work for PRs too or is tag-ur-it only designed for issues only?

@damccorm
Copy link
Owner

As of now its for issues only. In theory I don't think it would be that hard to extend for PRs, I think it would probably be changes here:

if (github.context.payload.action !== 'opened') {
, and here:
if (!github.context.payload.issue) {

along with some changes to the inputs to make sure we can maintain back compat (ideally the default behavior would be to enable it only on issues, with the option to enable it on PRs or both).

I don't have the bandwidth to take this on right now, but would be happy to review a pr :)

@damccorm damccorm added help wanted Extra attention is needed good first issue Good for newcomers labels Jun 17, 2020
@kdeldycke
Copy link
Author

Thanks @damccorm for the quick feedback! Supporting PRs is indeed doable. If I find the bandwidth too I'll be happy to contribute! :)

@damccorm
Copy link
Owner

Lets leave this one open to track this as an enhancement!

@damccorm damccorm reopened this Jun 17, 2020
@damccorm damccorm added enhancement New feature or request and removed question Further information is requested labels Jun 17, 2020
@kdeldycke
Copy link
Author

OK no worries. I was just trying to not pollute your backlog! :)

@kdeldycke kdeldycke reopened this Jun 17, 2020
@jenlampton
Copy link

I'm confused as to where the labels get assigned. I'm looking for a solution to add the label of "has pull request" to an issue that, obviously, has a pull request.

Is this enhancement asking to add labels onto PRs, or to add lables onto issues when PRs are filed?

@kdeldycke
Copy link
Author

@jenlampton The former. So this feature request is about "teaching" tag-ur-it to add labels to PRs, the same way it currently does with issues.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
damccorms problem now enhancement New feature or request good first issue Good for newcomers help wanted Extra attention is needed plz help
Projects
None yet
Development

No branches or pull requests

3 participants