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

<p>This passed an OWG circular resolution so in principle we are good to go subject to technical review.</p>
<p>Other than the inline comments I have made I think my only outstanding question is how we want to refer to this - principally whether to include "OSRM" in the name or just refer to it as FOSSGIS?</p>
<p>Previously we have mixed both styles to some extent, although in some cases like Graphhopper the provider and the technology are the same, but in other cases (like MapQuest) we gave used the provider name and in others (like OSRM) we have used the technology name.</p><hr>

<p>In <a href="https://github.com/openstreetmap/openstreetmap-website/pull/2063#discussion_r246871675">config/locales/en.yml</a>:</p>
<pre style='color:#555'>> @@ -2441,13 +2441,15 @@ en:
     directions:
       ascend: "Ascend"
       engines:
+        fossgis_osrm_bike: "Bicycle (FOSSGIS OSRM)"
+        fossgis_osrm_car: "Car (FOSSGIS OSRM))"
+        fossgis_osrm_foot: "Foot (FOSSGIS OSRM))"
</pre>
<p>We seem to have a superfluity of parentheses here...</p>

<hr>

<p>In <a href="https://github.com/openstreetmap/openstreetmap-website/pull/2063#discussion_r246871962">config/example.application.yml</a>:</p>
<pre style='color:#555'>> @@ -102,6 +102,7 @@ defaults: &defaults
   graphhopper_url: "https://graphhopper.com/api/1/route"
   mapquest_directions_url: "https://open.mapquestapi.com/directions/v2/route"
   osrm_url: "https://router.project-osrm.org/route/v1/driving/"
</pre>
<p>This line can be removed I think?</p>

<hr>

<p>In <a href="https://github.com/openstreetmap/openstreetmap-website/pull/2063#discussion_r246872689">app/assets/javascripts/index/directions/fossgis.js</a>:</p>
<pre style='color:#555'>> @@ -119,9 +119,9 @@ function OSRMEngine() {
           namedRoad = false;
         }
 
-        if (step.maneuver.type.match(/^exit (rotary|roundabout)$/)) {
+        if (step.maneuver.type.match(/exit (rotary|roundabout)/)) {
</pre>
<p>What's the reason for removing the anchors from this, and the other matches? Is this server producing different directions that have something else before and after?</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/2063#pullrequestreview-191351097">view it on GitHub</a>, or <a href="https://github.com/notifications/unsubscribe-auth/ABWnLQx1AVed0LcIjxxMVg7jkzKa1TrAks5vB4gngaJpZM4YeThw">mute the thread</a>.<img src="https://github.com/notifications/beacon/ABWnLSb9ME2YCRn6eSt3PLdyGun_O5Csks5vB4gngaJpZM4YeThw.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://github.githubassets.com/images/email/message_cards/header.png","avatar_image_url":"https://github.githubassets.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 requested changes on #2063"}],"action":{"name":"View Pull Request","url":"https://github.com/openstreetmap/openstreetmap-website/pull/2063#pullrequestreview-191351097"}}}</script>
<script type="application/ld+json">[
{
"@context": "http://schema.org",
"@type": "EmailMessage",
"potentialAction": {
"@type": "ViewAction",
"target": "https://github.com/openstreetmap/openstreetmap-website/pull/2063#pullrequestreview-191351097",
"url": "https://github.com/openstreetmap/openstreetmap-website/pull/2063#pullrequestreview-191351097",
"name": "View Pull Request"
},
"description": "View this Pull Request on GitHub",
"publisher": {
"@type": "Organization",
"name": "GitHub",
"url": "https://github.com"
}
}
]</script>