Project

Profile

Help

Issue #4441

closed

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

Added by daviddavis about 5 years ago. Updated over 3 years ago.

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.

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

Actions #1

Updated by CodeHeeler about 5 years ago

  • Triaged changed from No to Yes
Actions #2

Updated by bmbouter almost 5 years ago

  • Tags deleted (Pulp 3)
Actions #3

Updated by daviddavis over 3 years ago

  • Status changed from NEW to CLOSED - CURRENTRELEASE

Also available in: Atom PDF