Git with a cup of tea, painless self-hosted git service Mirror for internal git.with.parts use https://git.with.parts

You can not select more than 25 topics Topics must start with a letter or number, can include dashes ('-') and can be up to 35 characters long.

kolaente 085f717529 feat: notify doers of a merge when automerging (#21553) ... I found myself wondering whether a PR I scheduled for automerge was actually merged. It was, but I didn't receive a mail notification for it - that makes sense considering I am the doer and usually don't want to receive such notifications. But ideally I want to receive a notification when a PR was merged because I scheduled it for automerge. This PR implements exactly that. The implementation works, but I wonder if there's a way to avoid passing the "This PR was automerged" state down so much. I tried solving this via the database (checking if there's an automerge scheduled for this PR when sending the notification) but that did not work reliably, probably because sending the notification happens async and the entry might have already been deleted. My implementation might be the most straightforward but maybe not the most elegant. Signed-off-by: Andrew Thornton <art27@cantab.net> Co-authored-by: Lauris BH <lauris@nix.lv> Co-authored-by: Andrew Thornton <art27@cantab.net> Co-authored-by: Lunny Xiao <xiaolunwen@gmail.com>		2 years ago
..
mail.go	feat: notify doers of a merge when automerging (#21553)	2 years ago
mail_comment.go	Add generic set type (#21408)	2 years ago
mail_issue.go	feat: notify doers of a merge when automerging (#21553)	2 years ago
mail_release.go	Add team member invite by email (#20307)	2 years ago
mail_repo.go	Refactor `i18n` to `locale` (#20153)	2 years ago
mail_team_invite.go	Add team member invite by email (#20307)	2 years ago
mail_test.go	Move some files into models' sub packages (#20262)	2 years ago
mailer.go	Replace all instances of fmt.Errorf(%v) with fmt.Errorf(%w) (#21551)	2 years ago
mailer_test.go	refactoring nits (#18188)	3 years ago
main_test.go	Use a struct as test options (#19393)	3 years ago