Project

Profile

Help

Issue #4441

Our "Required PR" syntax is munged by Github and causes confusion

Added by daviddavis 8 months ago. Updated 6 months ago.

Status:
NEW
Priority:
Normal
Assignee:
-
Category:
-
Sprint/Milestone:
-
Start date:
Due date:
Severity:
2. Medium
Version:
Platform Release:
Blocks Release:
OS:
Backwards Incompatible:
No
Triaged:
Yes
Groomed:
No
Sprint Candidate:
No
Tags:
QA Contact:
Complexity:
Smash Test:
Verified:
No
Verification Required:
No
Sprint:

Description

When you add a required pr to your commit message0, 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.

[0] https://docs.pulpproject.org/en/3.0/nightly/contributing/continuous-integration.html#requiring-other-pull-requests

History

#1 Updated by CodeHeeler 8 months ago

  • Triaged changed from No to Yes

#2 Updated by bmbouter 6 months ago

  • Tags deleted (Pulp 3)

Please register to edit this issue

Also available in: Atom PDF