Making WordPress.org

Opened 3 years ago

Closed 3 years ago

Last modified 3 years ago

#5644 closed enhancement (fixed)

Exclude codecov bot comments from Trac

Reported by: desrosj's profile desrosj Owned by: dd32's profile dd32
Milestone: Priority: normal
Component: Trac Keywords:
Cc:

Description

Under some conditions, the codecov bot will post a comment on a PR with summarizing the impact the PR will have on code coverage.

Copying these comments over to Trac is not necessary, so the codecov[bot] user can be added to the ignore list.

https://github.com/WordPress/wordpress-develop/pull/1049#issuecomment-786340007

Related #5551.

Change History (4)

#1 follow-up: @dd32
3 years ago

Should we just ignore all bots flagged as such?

'[bot]' === substr( $payload->comment->user->login, -5 ) // All bot users.

#2 @dd32
3 years ago

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

In 10735:

Trac: PRs: Ignore the codecov[bot] bot when syncing comments.

Fixes #5644.

#3 in reply to: ↑ 1 @desrosj
3 years ago

Replying to dd32:

Should we just ignore all bots flagged as such?

'[bot]' === substr( $payload->comment->user->login, -5 ) // All bot users.

That's probably a good idea. I can't really think of a time when we would want a bot comment to be synced to Trac. Bots are usually posting something that is contextual to the PR and not so much the ticket or problem being solved.

#4 @dd32
3 years ago

In 10736:

Trac: PRs: Ignore all ....[bot] bot users when syncing comments.

Fixes #5644.

Note: See TracTickets for help on using tickets.