jaywink / matrix-alertmanager

A bot to receive Alertmanager webhook events and forward them to chosen rooms.
MIT License
76 stars 22 forks source link

Richer alerts; improved docs; easier testing #39

Open jinnko opened 1 year ago

jinnko commented 1 year ago

You may not want to merge this, but thought I'd share just in case.

From the individual commit message:

jinnko commented 11 months ago

@jelmer - yes - I intentionally make commits each with their own goal - so you're right they could be their own PRs. If you merge without squashing they will continue to be discrete changes in the history.

As they're all applied on my forked master branch - I guess the alternative would be to cherry pick each commit into it's own branch and raise multiple PRs - but the end result would be the same.

jelmer commented 11 months ago

@jinnko makes sense - FWIM I'm not a committer to matrix-alertmanager, just an interested contributor