<p></p>
<p>I'm going to close this PR since I've also closed the original issue.</p>
<p>Just to help you with future PRs, I'll mention a couple of things that I noticed:</p>
<ul>
<li>This PR removed the Export link, without providing a replacement or explanation. I'm not sure if that was intended?</li>
<li>The PR contains too many 'fixup' commits. In general PRs can contain multiple commits (and I prefer if they do) but each commit should be a logical group of work, rather than a blow-by-blow record of winning the battle with rubocop :-) My recommendation is to use a combination of <code>git rebase -i</code> (the 'fixup' option is great for combining such fixup commits into the original commit, and I use this every week)  and then force-push the branch to update the PR with your preferred list of commits.</li>
</ul>

<p style="font-size:small;-webkit-text-size-adjust:none;color:#666;">—<br />You are receiving this because you are subscribed to this thread.<br />Reply to this email directly, <a href="https://github.com/openstreetmap/openstreetmap-website/pull/3086#issuecomment-780452494">view it on GitHub</a>, or <a href="https://github.com/notifications/unsubscribe-auth/AAK2OLOSI2AWY4PQSXMO673S7OJSVANCNFSM4XHVOP5A">unsubscribe</a>.<img src="https://github.com/notifications/beacon/AAK2OLP27BAMA4I7DB5IZ7DS7OJSVA5CNFSM4XHVOP5KYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOF2CMFDQ.gif" height="1" width="1" alt="" /></p>
<script type="application/ld+json">[
{
"@context": "http://schema.org",
"@type": "EmailMessage",
"potentialAction": {
"@type": "ViewAction",
"target": "https://github.com/openstreetmap/openstreetmap-website/pull/3086#issuecomment-780452494",
"url": "https://github.com/openstreetmap/openstreetmap-website/pull/3086#issuecomment-780452494",
"name": "View Pull Request"
},
"description": "View this Pull Request on GitHub",
"publisher": {
"@type": "Organization",
"name": "GitHub",
"url": "https://github.com"
}
}
]</script>