<p></p>
<p><b>@tordans</b> commented on this pull request.</p>
<p dir="auto">I think this is a good template that will make thinks better for some PRs. I certainly does not harm the process, because one can always remove all this very easily when creating a PR that does not fit this schema.</p>
<p dir="auto">I would also remove the risk section. And make the intro even more explicit on the PR section which is where all the "do squash, do rebase, no merge commits" part is written.</p><hr>
<p>In <a href="https://github.com/openstreetmap/openstreetmap-website/pull/4919#discussion_r1649895559">.github/pull_request_template.md</a>:</p>
<pre style='color:#555'>> @@ -0,0 +1,10 @@
+<!--Please read the contributing guidelines before making a PR: https://github.com/openstreetmap/openstreetmap-website/blob/master/CONTRIBUTING.md-->
</pre>
⬇️ Suggested change
<pre style="color: #555">-<!--Please read the contributing guidelines before making a PR: https://github.com/openstreetmap/openstreetmap-website/blob/master/CONTRIBUTING.md-->
+<!--
+Please read the contributing guidelines before making a PR:
+ https://github.com/openstreetmap/openstreetmap-website/blob/master/CONTRIBUTING.md
+
+Especially the section on how to present PRs:
+ https://github.com/openstreetmap/openstreetmap-website/blob/master/CONTRIBUTING.md#pull-requests
+-->
</pre>
<hr>
<p>In <a href="https://github.com/openstreetmap/openstreetmap-website/pull/4919#discussion_r1649895571">.github/pull_request_template.md</a>:</p>
<pre style='color:#555'>> +
+### Risks
+<!--Is this a low, medium, or high risk change? Explain why.-->
</pre>
⬇️ Suggested change
<pre style="color: #555">-
-### Risks
-<!--Is this a low, medium, or high risk change? Explain why.-->
</pre>
<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/pull/4919#pullrequestreview-2134062525">view it on GitHub</a>, or <a href="https://github.com/notifications/unsubscribe-auth/AAK2OLOH7ZYMY3HYM7SUYN3ZIZE5FAVCNFSM6AAAAABJXTDRV6VHI2DSMVQWIX3LMV43YUDVNRWFEZLROVSXG5CSMV3GSZLXHMZDCMZUGA3DENJSGU">unsubscribe</a>.<br />You are receiving this because you are subscribed to this thread.<img src="https://github.com/notifications/beacon/AAK2OLIRGGVWMM2OL7VXQXTZIZE5FA5CNFSM6AAAAABJXTDRV6WGG33NNVSW45C7OR4XAZNRKB2WY3CSMVYXKZLTORJGK5TJMV32UY3PNVWWK3TUL5UWJTT7GM232.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/pull/4919/review/2134062525</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/pull/4919#pullrequestreview-2134062525",
"url": "https://github.com/openstreetmap/openstreetmap-website/pull/4919#pullrequestreview-2134062525",
"name": "View Pull Request"
},
"description": "View this Pull Request on GitHub",
"publisher": {
"@type": "Organization",
"name": "GitHub",
"url": "https://github.com"
}
}
]</script>