tag-ur-it icon indicating copy to clipboard operation
tag-ur-it copied to clipboard

Pull-requests support or issues-only?

Open kdeldycke opened this issue 4 years ago • 6 comments

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?

kdeldycke avatar Jun 17 '20 09:06 kdeldycke

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: https://github.com/damccorm/tag-ur-it/blob/4cf02d3b4ae7f248713b11d35a75522c10a8deed/src/index.ts#L13, and here: https://github.com/damccorm/tag-ur-it/blob/4cf02d3b4ae7f248713b11d35a75522c10a8deed/src/index.ts#L18

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 avatar Jun 17 '20 11:06 damccorm

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

kdeldycke avatar Jun 17 '20 12:06 kdeldycke

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

damccorm avatar Jun 17 '20 12:06 damccorm

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

kdeldycke avatar Jun 17 '20 13:06 kdeldycke

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?

jenlampton avatar Oct 01 '20 22:10 jenlampton

@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.

kdeldycke avatar Oct 01 '20 22:10 kdeldycke