Actions
Issue #4441
closedOur "Required PR" syntax is munged by Github and causes confusion
Status:
CLOSED - CURRENTRELEASE
Priority:
Normal
Assignee:
-
Category:
-
Sprint/Milestone:
-
Start date:
Due date:
Estimated time:
Severity:
2. Medium
Version:
Platform Release:
OS:
Triaged:
Yes
Groomed:
No
Sprint Candidate:
No
Tags:
Sprint:
Quarter:
Description
When you add a required pr to your commit message[0], Github will show "pulp/pulp_file#123" for "https://github.com/pulp/pulp_file/pull/123". This creates a lot of confusion as developers will sometimes do "Required PR: pulp/pulp_file#123".
Not sure about the best solution. One option would be to accept the syntax "Required PR: pulp/pulp_file#123" while another option would be to change the syntax.
Actions