Making WordPress.org

Opened 22 months ago

Closed 22 months ago

Last modified 19 months ago

#5057 closed defect (fixed)

Remove comments from pull request description

Reported by: isabel_brison Owned by: dd32
Milestone: Priority: normal
Component: Trac Keywords:


Following on from https://core.trac.wordpress.org/ticket/49508, PR descriptions that get synced to core Trac currently include all the boilerplate comments, which makes the descriptions hard to read.

It would be good to filter out those comments from the payload that comes from the GitHub API.

Change History (3)

#1 @dd32
22 months ago

Looks like there's two things that needs doing here for the best experience:

  • Strip HTML comments
  • Strip ---\r\n\*\*.+\*\*$ from the response to remove the code-review last-line in a way that will preserve anything else added to the desc.

#2 @dd32
22 months ago

  • Owner set to dd32
  • Resolution set to fixed
  • Status changed from new to closed

In 9539:

Trac: Github PRs: Filter out HTML comments and standard code-review-only boilerplate from PR descriptions before posting them to Trac.

Props isabel_brison for the suggestion.
Fixes #5057.

#3 @dd32
19 months ago

In 9877:

Trac: Github PRs: Actually filter out HTML comments.

This commit message is longer than the fix.

Amends [9539].
See #5057.
Fixes #5213.

Note: See TracTickets for help on using tickets.