<div style="display: flex; flex-wrap: wrap; white-space: pre-wrap; align-items: center; "><img height="20" width="20" style="border-radius:50%; margin-right: 4px;" decoding="async" src="https://avatars.githubusercontent.com/u/360803?s=20&v=4" /><strong>gravitystorm</strong> left a comment <a href="https://github.com/openstreetmap/openstreetmap-website/pull/5988#issuecomment-2859302530">(openstreetmap/openstreetmap-website#5988)</a></div>
<blockquote>
<p dir="auto">You can just bump that limit - anything in <code class="notranslate">.rubocop_todo.yml</code> is just there to allow things that we haven't got around to doing anything yet and for those metrics ones at least it's not at all clear what if anything we even want to do.</p>
</blockquote>
<p dir="auto">Well.... they are there in order to discourage the situation from getting even worse. In many circumstances it's pragmatic to bump them up a bit (e.g. adding one more line to a 250-line class) but if we keep hitting these limits or if we blow past them in one PR, it's an indication that we should be refactoring.</p>
<p dir="auto">For the metrics, I tend to find that the rubocop defaults are too small, our current values are way too large, and we could agree on a custom limit somewhere between the two (with a bit of refactoring for the worst offending files).</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/pull/5988#issuecomment-2859302530">view it on GitHub</a>, or <a href="https://github.com/notifications/unsubscribe-auth/AAK2OLLYHQPMIJBIYBAX5A325I24PAVCNFSM6AAAAAB4SSMU4WVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDQNJZGMYDENJTGA">unsubscribe</a>.<br />You are receiving this because you are subscribed to this thread.<img src="https://github.com/notifications/beacon/AAK2OLNJ3D276XAMHKAC4CD25I24PA5CNFSM6AAAAAB4SSMU4WWGG33NNVSW45C7OR4XAZNMJFZXG5LFINXW23LFNZ2KUY3PNVWWK3TUL5UWJTVKNV7IE.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/5988/c2859302530</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/5988#issuecomment-2859302530",
"url": "https://github.com/openstreetmap/openstreetmap-website/pull/5988#issuecomment-2859302530",
"name": "View Pull Request"
},
"description": "View this Pull Request on GitHub",
"publisher": {
"@type": "Organization",
"name": "GitHub",
"url": "https://github.com"
}
}
]</script>