[openstreetmap/openstreetmap-website] Automate some code review comments (Issue #4912)

[Andy Allan]

There's a few common code review comments that could be automated (or in the meantime, community members are welcome to jump in and help out the maintainers with these reviews!):

* Any changes to locale files, other than `en.yml`, should be flagged and the PR amended
* Any PRs that have a merge commit should be flagged and the PR amended

In both cases, the response is more-or-less the same content each time, usually with a link to the right place in our documentation. 

We could automate this, which would bring a few benefits, namely
* Faster feedback cycle. Contributors would get the message straight away, while they might still be working on their PR, and not several hours/days later.
* Less work for maintainers - we're overstretched already, so it's one less thing to take care of.
* More polished responses - sometimes we're a bit pressed for time, or forget the link to the documentation, or similar.
* More support for maintainers - like linters and tests, it could highlight things we might miss.

I've seen other projects using Danger for this, since it's able to access the state of the PR in git, i.e. it can reason about the changes, not just the end state (otherwise we'd use linters etc for that).

https://danger.systems/ruby/

Anyone fancy implementing these rules? Or any reasons not to?



-- 
Reply to this email directly or view it on GitHub:
https://github.com/openstreetmap/openstreetmap-website/issues/4912
You are receiving this because you are subscribed to this thread.

Message ID: <openstreetmap/openstreetmap-website/issues/4912 at github.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstreetmap.org/pipermail/rails-dev/attachments/20240619/3c7a5375/attachment.htm>