<p>What was the logic behind "fixing" the <code>AssignmentInCondition</code> offenses? It doesn't look like that would have triggered at the point so I'm guessing the issue was that moving them to the new helper would cause them to start triggering unless you excluded that?</p>
<p>For the record the reason I never fixed that is that I didn't see it as a major problem and I find the result extremely ugly :-(</p>
<p>That said there are a number of other places where I have similar feelings but have given in and held my nose and let rubocop have it's way so maybe we should do the same here - either that or just decide we don't like that rule and disable it globally.</p>

<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/2072#issuecomment-442175841">view it on GitHub</a>, or <a href="https://github.com/notifications/unsubscribe-auth/ABWnLdzPU-vhxO-vjIwdHZFLG0Gq1_r0ks5uzYtAgaJpZM4Y0ZKk">mute the thread</a>.<img src="https://github.com/notifications/beacon/ABWnLfXyJStmDM-WTfBbYHsegrh1W4PPks5uzYtAgaJpZM4Y0ZKk.gif" height="1" width="1" alt="" /></p>
<script type="application/json" data-scope="inboxmarkup">{"api_version":"1.0","publisher":{"api_key":"05dde50f1d1a384dd78767c55493e4bb","name":"GitHub"},"entity":{"external_key":"github/openstreetmap/openstreetmap-website","title":"openstreetmap/openstreetmap-website","subtitle":"GitHub repository","main_image_url":"https://assets-cdn.github.com/images/email/message_cards/header.png","avatar_image_url":"https://assets-cdn.github.com/images/email/message_cards/avatar.png","action":{"name":"Open in GitHub","url":"https://github.com/openstreetmap/openstreetmap-website"}},"updates":{"snippets":[{"icon":"PERSON","message":"@tomhughes in #2072: What was the logic behind \"fixing\" the `AssignmentInCondition` offenses? It doesn't look like that would have triggered at the point so I'm guessing the issue was that moving them to the new helper would cause them to start triggering unless you excluded that?\r\n\r\nFor the record the reason I never fixed that is that I didn't see it as a major problem and I find the result extremely ugly :-(\r\n\r\nThat said there are a number of other places where I have similar feelings but have given in and held my nose and let rubocop have it's way so maybe we should do the same here - either that or just decide we don't like that rule and disable it globally."}],"action":{"name":"View Pull Request","url":"https://github.com/openstreetmap/openstreetmap-website/pull/2072#issuecomment-442175841"}}}</script>
<script type="application/ld+json">[
{
"@context": "http://schema.org",
"@type": "EmailMessage",
"potentialAction": {
"@type": "ViewAction",
"target": "https://github.com/openstreetmap/openstreetmap-website/pull/2072#issuecomment-442175841",
"url": "https://github.com/openstreetmap/openstreetmap-website/pull/2072#issuecomment-442175841",
"name": "View Pull Request"
},
"description": "View this Pull Request on GitHub",
"publisher": {
"@type": "Organization",
"name": "GitHub",
"url": "https://github.com"
}
},
{
"@type": "MessageCard",
"@context": "http://schema.org/extensions",
"hideOriginalBody": "false",
"originator": "AF6C5A86-E920-430C-9C59-A73278B5EFEB",
"title": "Re: [openstreetmap/openstreetmap-website] Show tel: links for multiple phone numbers separated by ; (#2072)",
"sections": [
{
"text": "",
"activityTitle": "**Tom Hughes**",
"activityImage": "https://assets-cdn.github.com/images/email/message_cards/avatar.png",
"activitySubtitle": "@tomhughes",
"facts": [

]
}
],
"potentialAction": [
{
"name": "Add a comment",
"@type": "ActionCard",
"inputs": [
{
"isMultiLine": true,
"@type": "TextInput",
"id": "IssueComment",
"isRequired": false
}
],
"actions": [
{
"name": "Comment",
"@type": "HttpPOST",
"target": "https://api.github.com",
"body": "{\n\"commandName\": \"IssueComment\",\n\"repositoryFullName\": \"openstreetmap/openstreetmap-website\",\n\"issueId\": 2072,\n\"IssueComment\": \"{{IssueComment.value}}\"\n}"
}
]
},
{
"name": "Close pull request",
"@type": "HttpPOST",
"target": "https://api.github.com",
"body": "{\n\"commandName\": \"PullRequestClose\",\n\"repositoryFullName\": \"openstreetmap/openstreetmap-website\",\n\"pullRequestId\": 2072\n}"
},
{
"targets": [
{
"os": "default",
"uri": "https://github.com/openstreetmap/openstreetmap-website/pull/2072#issuecomment-442175841"
}
],
"@type": "OpenUri",
"name": "View on GitHub"
},
{
"name": "Unsubscribe",
"@type": "HttpPOST",
"target": "https://api.github.com",
"body": "{\n\"commandName\": \"MuteNotification\",\n\"threadId\": 416387748\n}"
}
],
"themeColor": "26292E"
}
]</script>