<p></p>
<p dir="auto">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!):</p>
<ul dir="auto">
<li>Any changes to locale files, other than <code class="notranslate">en.yml</code>, should be flagged and the PR amended</li>
<li>Any PRs that have a merge commit should be flagged and the PR amended</li>
</ul>
<p dir="auto">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.</p>
<p dir="auto">We could automate this, which would bring a few benefits, namely</p>
<ul dir="auto">
<li>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.</li>
<li>Less work for maintainers - we're overstretched already, so it's one less thing to take care of.</li>
<li>More polished responses - sometimes we're a bit pressed for time, or forget the link to the documentation, or similar.</li>
<li>More support for maintainers - like linters and tests, it could highlight things we might miss.</li>
</ul>
<p dir="auto">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).</p>
<p dir="auto"><a href="https://danger.systems/ruby/" rel="nofollow">https://danger.systems/ruby/</a></p>
<p dir="auto">Anyone fancy implementing these rules? Or any reasons not to?</p>

<p style="font-size:small;-webkit-text-size-adjust:none;color:#666;">—<br />Reply to this email directly, <a href="https://github.com/openstreetmap/openstreetmap-website/issues/4912">view it on GitHub</a>, or <a href="https://github.com/notifications/unsubscribe-auth/AAK2OLKBC422CINW3DLUS2LZIGKI5AVCNFSM6AAAAABJSGZGOKVHI2DSMVQWIX3LMV43ASLTON2WKOZSGM3DENJWGA4TSNQ">unsubscribe</a>.<br />You are receiving this because you are subscribed to this thread.<img src="https://github.com/notifications/beacon/AAK2OLK66ZH6Q3R2I756RILZIGKI5A5CNFSM6AAAAABJSGZGOKWGG33NNVSW45C7OR4XAZNFJFZXG5LFVJRW63LNMVXHIX3JMTHIZUOR4Q.gif" height="1" width="1" alt="" /><span style="color: transparent; font-size: 0; display: none; visibility: hidden; overflow: hidden; opacity: 0; width: 0; height: 0; max-width: 0; max-height: 0; mso-hide: all">Message ID: <span><openstreetmap/openstreetmap-website/issues/4912</span><span>@</span><span>github</span><span>.</span><span>com></span></span></p>
<script type="application/ld+json">[
{
"@context": "http://schema.org",
"@type": "EmailMessage",
"potentialAction": {
"@type": "ViewAction",
"target": "https://github.com/openstreetmap/openstreetmap-website/issues/4912",
"url": "https://github.com/openstreetmap/openstreetmap-website/issues/4912",
"name": "View Issue"
},
"description": "View this Issue on GitHub",
"publisher": {
"@type": "Organization",
"name": "GitHub",
"url": "https://github.com"
}
}
]</script>