cycleway value should fall back to cycleway:both if available#6180
cycleway value should fall back to cycleway:both if available#6180Nate-Wessel wants to merge 3 commits intoProject-OSRM:masterfrom
cycleway value should fall back to cycleway:both if available#6180Conversation
|
@Nate-Wessel can you include |
|
Alas, I'd be happy to contribute further but I'm really not sure what I'm looking at with the feature tests there. I think I'd be as likely to break something as fix it. |
|
closing stale PR. Reopen if still relevant. |
I would like to reopen this, but don't see the option to do so. How do I reopen? |
|
Reopened |
|
I guess all we'd need with this PR would be a test case that covers the change. |
|
I'll be happy to add a test case if you can point me toward how I would do that. Is there any documentation? |
|
Have a look at testing docs. The section where it says 'Cucumber' is probably the most interesting one. |
|
Friendly ping on adding a cucumber test to this. It seems this could be covered by a |
|
This PR seems to be stale. Is it still relevant? |
|
we should be able to shepherd this change in, finally, after CI passes. |
|
This PR seems to be stale. Is it still relevant? |
Yes. |
|
@Nate-Wessel thank your contribution I added a test in #7314 |
fixes #6179