<p></p>
<p><b>@gravitystorm</b> requested changes on this pull request.</p>

<hr>

<p>In <a href="https://github.com/openstreetmap/openstreetmap-website/pull/2889#discussion_r509328333">app/views/oauth_clients/index.html.erb</a>:</p>
<pre style='color:#555'>> @@ -15,7 +15,7 @@
   </thead>
   <% @tokens.each do |token| %>
     <tr>
-      <td><%= link_to token.client_application.name, token.client_application.url %></td>
+      <td><%= link_to token.client_application.name, token.client_application.url, target: "_blank" %></td>
</pre>
<p>We rarely use <code>target: "_blank"</code> for links - any particular reason to add it to this one?</p>
<p>Also this line causes the CI failures for both hash rocket syntax, and noopener when using target: "_blank"</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/2889#pullrequestreview-513740696">view it on GitHub</a>, or <a href="https://github.com/notifications/unsubscribe-auth/AAK2OLLAPVQQAIGM27MEMNDSL3UYDANCNFSM4SUL4G6Q">unsubscribe</a>.<img src="https://github.com/notifications/beacon/AAK2OLP33SA6GO4625IT7XDSL3UYDA5CNFSM4SUL4G62YY3PNVWWK3TUL52HS4DFWFIHK3DMKJSXC5LFON2FEZLWNFSXPKTDN5WW2ZLOORPWSZGOD2PQ7GA.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/2889#pullrequestreview-513740696",
"url": "https://github.com/openstreetmap/openstreetmap-website/pull/2889#pullrequestreview-513740696",
"name": "View Pull Request"
},
"description": "View this Pull Request on GitHub",
"publisher": {
"@type": "Organization",
"name": "GitHub",
"url": "https://github.com"
}
}
]</script>