GitHub integration: Sync pull request description

[noisysocks]

It would be useful to sync the pull request's description to the linked Trac ticket. Descriptions often contain important information such as testing instructions or implementation details, and can be considered the first comment in a pull request.

[noisysocks]

The PR description / first comment

[dd32]

The only pitfall I can see here is that the Markdown syntax between PR's and Trac comments differ slightly, so some things will come over a little scrambled. But for most basic things it should work just fine.

I'd probably suggest we enable the email notifications for the "PR has been linked to this ticket" comment if it contains such a comment too.

[dd32]

In 9531:

Trac: Github PRs: Include the PR bodytext in the Trac notification.

This should help include some more context in the PRs and also make it easier to keep all pertitant information in one place (Trac).

See #5054.

[dd32]

r9531 seems to work in my testing, but let's see what it actually looks like once some production PRs are created..

I didn't enable Email notifications for the comment, but that's the obvious next thing before this is closed (Assuming this hasn't broken Trac :) )

[dd32]

In 9532:

Trac: Github PRs: Enable email notifications for the new-PR-linked comment.

See #5054.

[dd32]

Replying to dd32:

In 9532:

Trac: Github PRs: Enable email notifications for the new-PR-linked comment.

See #5054.

Doesn't look like this worked.. I don't see any notifications for ​https://core.trac.wordpress.org/ticket/49495#comment:4

[dd32]

#5143 was marked as a duplicate.

[dd32]

Replying to dd32:

Replying to dd32:

In 9532:

Trac: Github PRs: Enable email notifications for the new-PR-linked comment.

See #5054.

Doesn't look like this worked.. I don't see any notifications for ​https://core.trac.wordpress.org/ticket/49495#comment:4

Testing against my own Trac instance, it looks like the RPC call is being processed correctly.

[garrett-eclipse]

@dd32 just an FYI I'm not seeing notifications from PRs attached to Trac. If people don't comment on the ticket after making the PR then it's silent and watchers aren't aware there's a new patch.
Specifically speaking on the PR#322 on this ticket #43994
*Commenting here as I followed the duplicate trail to this ticket.

A second thought is there's not a comment number that's linked to the comment so you can't share them or point them out directly like you can for other comments in Trac.

[dd32]

just an FYI I'm not seeing notifications from PRs attached to Trac. If people don't comment on the ticket after making the PR then it's silent and watchers aren't aware there's a new patch.

Correct. Email notifications from Trac don't work, no idea why, the RPC calls are correct but Trac isn't sending notifications.

[dd32]

I'm going to re-open #5143 and close this ticket as fixed.