<p>This is really hard to review because you seem to have made a lot of non-essential changes to the tests.</p>
<p>It would probably have been better to leave the existing tests (bar the changes needed to adapt to the API change) and add new tests for the semicolon separated lists to the end so that comparison tools are able to show the changes more clearly.</p>
<p>I think I'm right in saying that the only function change is that <code>/</code> is no longer allowed in numbers? Why did you choose to do that?</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/2071#issuecomment-441443519">view it on GitHub</a>, or <a href="https://github.com/notifications/unsubscribe-auth/ABWnLccj6XKCx6nF1BkC7olwFxtR6-mYks5uyqWhgaJpZM4Yxjxu">mute the thread</a>.<img src="https://github.com/notifications/beacon/ABWnLdT7wANqMhwZNIEoCZtAB1-WnZx1ks5uyqWhgaJpZM4Yxjxu.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 #2071: This is really hard to review because you seem to have made a lot of non-essential changes to the tests.\r\n\r\nIt would probably have been better to leave the existing tests (bar the changes needed to adapt to the API change) and add new tests for the semicolon separated lists to the end so that comparison tools are able to show the changes more clearly.\r\n\r\nI think I'm right in saying that the only function change is that `/` is no longer allowed in numbers? Why did you choose to do that?"}],"action":{"name":"View Pull Request","url":"https://github.com/openstreetmap/openstreetmap-website/pull/2071#issuecomment-441443519"}}}</script>
<script type="application/ld+json">[
{
"@context": "http://schema.org",
"@type": "EmailMessage",
"potentialAction": {
"@type": "ViewAction",
"target": "https://github.com/openstreetmap/openstreetmap-website/pull/2071#issuecomment-441443519",
"url": "https://github.com/openstreetmap/openstreetmap-website/pull/2071#issuecomment-441443519",
"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 on multiple phone numbers separated by ; (#2071)",
"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\": 2071,\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\": 2071\n}"
},
{
"targets": [
{
"os": "default",
"uri": "https://github.com/openstreetmap/openstreetmap-website/pull/2071#issuecomment-441443519"
}
],
"@type": "OpenUri",
"name": "View on GitHub"
},
{
"name": "Unsubscribe",
"@type": "HttpPOST",
"target": "https://api.github.com",
"body": "{\n\"commandName\": \"MuteNotification\",\n\"threadId\": 415644782\n}"
}
],
"themeColor": "26292E"
}
]</script>