Opened 3 years ago

Last modified 3 years ago

#63821 assigned defect

macportsbot: exceeds maximum number of mentions per comment

Reported by: chrstphrchvz (Christopher Chavez) Owned by: admin@…
Priority: Normal Milestone:
Component: server/hosting Version:
Keywords: Cc: l2dy (Zero King)
Port:

Description

GitHub appears to limit the number of mentioned handles per comment to 50; so in comments such as https://github.com/macports/macports-ports/pull/12807#issuecomment-961405770, I would think only the first 50 maintainers were successfully pinged. Should macportsbot split up notification comments to limit the number of notified maintainers per comment?

Change History (3)

comment:1 Changed 3 years ago by ryandesign (Ryan Carsten Schmidt)

Cc: l2dy added

I believe the code is here: https://github.com/macports/mpbot-github; I know nothing about it.

comment:2 Changed 3 years ago by l2dy (Zero King)

I was worried about contributors who are not familiar with Git open a pull request with a lot of rebased commits and accidentally ping a lot of maintainers, but this shouldn't be a problem now.

We welcome contributions to the bot, related code is https://github.com/macports/mpbot-github/blob/a79d1c86e4e9cc5daa284107132140f9c52b6cd7/pr/webhook/pull_request.go#L119-L132.

comment:3 Changed 3 years ago by chrstphrchvz (Christopher Chavez)

Thanks for pointing me to the relevant code. I’m not familiar with Go, but here is a completely untested attempt: https://github.com/macports/mpbot-github/pull/8

Note: See TracTickets for help on using tickets.