Context Navigation

← Previous Ticket
Next Ticket →

#5080 closed enhancement (fixed)

When a PR is created for a Trac ticket apply 'has-patch'

Reported by:	garrett-eclipse	Owned by:	dd32
Priority:	normal	Milestone:
Component:	Trac	Keywords:
Cc:

Description

Hello,

The new PR syncing is awesome. Might I suggest it trigger some keywords on Trac.
At minimum am thinking it should add has-patch, but possibly auto-add needs-testing.
And potentially have it remove needs-patch and needs-refresh.

*Also may not be possible but if unit tests are part of the PR maybe add has-unit-tests.

Thanks

Change History (3)

This ticket was mentioned in Slack in #meta by tellyworth. View the logs.

6 years ago

#2 @dd32
6 years ago

Owner set to dd32
Resolution → fixed
Status new → closed

In 9900:

Trac: When a PR is added to a ticket, toggle the has-patch/needs-patch/needs-refresh and needs-unit-tests/has-unit-tests if it touches a tests-looking directory.

Fixes #5080.

#3 @garrett-eclipse
6 years ago

You rock @dd32 thank you, I'll keep my eyes open for incoming PRs to test it out in the wild.

Note: See TracTickets for help on using tickets.

Download in other formats:

Making WordPress.org