cycleway value should fall back to cycleway:both if available

[Nate-Wessel]

fixes #6179

[mjjbell]

@Nate-Wessel can you include cycleway:both in the feature tests to ensure it also has coverage?

[Nate-Wessel]

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.

[DennisOSRM]

closing stale PR. Reopen if still relevant.

[Nate-Wessel]

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?

[DennisOSRM]

Reopened

[DennisOSRM]

I guess all we'd need with this PR would be a test case that covers the change.

[Nate-Wessel]

I'll be happy to add a test case if you can point me toward how I would do that. Is there any documentation?

[DennisOSRM]

Have a look at testing docs. The section where it says 'Cucumber' is probably the most interesting one.

[DennisOSRM]

Friendly ping on adding a cucumber test to this. It seems this could be covered by a @routability test.

[github-actions]

This PR seems to be stale. Is it still relevant?

[DennisOSRM]

we should be able to shepherd this change in, finally, after CI passes.

[github-actions]

This PR seems to be stale. Is it still relevant?

[Nate-Wessel]

This PR seems to be stale. Is it still relevant?

Yes.