WordPress.org
Get WordPress

Make WordPress Core

Opened 7 months ago

Closed 4 months ago

Last modified 4 months ago

#61567 closed defect (bug) (fixed)

Remove Trac ticket reminder from PR when one is added

Reported by: desrosj's profile desrosj Owned by: desrosj's profile desrosj
Milestone: 6.7 Priority: normal
Severity: normal Version:
Component: Build/Test Tools Keywords: has-patch
Focuses: Cc:

Description

#60129/[58092] added logic to comment on a PR to wordpress-develop when no Trac ticket URL is included (which is required).

When the PR is updated to include one, the comment should be removed to help limit the number of bot comments on a pull request.

Props @ocean90 for the suggestion.

Change History (6)

#1 @debarghyabanerjee
7 months ago

  • Keywords has-patch added; needs-patch removed

#2 @desrosj
5 months ago

  • Keywords changes-requested added

​@debarghyabanerjee commented on ​PR #7009:
5 months ago #3

Hi @desrosj, I have addressed the feedback and made the changes. I have also added the synchronize, reopened, & edited in the pull_request_track

#4 @desrosj
4 months ago

  • Keywords changes-requested removed
  • Owner set to desrosj
  • Status changed from new to reviewing

#5 @desrosj
4 months ago

  • Resolution set to fixed
  • Status changed from reviewing to closed

In 59169:

Build/Test Tools: Clean up PR comments after action is taken.

There’s now a number of automated comments left on pull requests to help contributors that a PR can easily be overridden with activity.

Some of the comments are only relevant until a specific action is taken. One such comment is for informing the PR author that a link to a Trac ticket is a requirement for considering any suggested changes.

This updates the pull request comment workflow to remove the comment once a link to a Trac ticket is properly included.

Props debarghyabanerjee.
Fixes #61567.

Note: See TracTickets for help on using tickets.