Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix missing | #1476

Merged
merged 1 commit into from
Jul 10, 2019
Merged

fix missing | #1476

merged 1 commit into from
Jul 10, 2019

Conversation

matkoniecz
Copy link
Member

@matkoniecz matkoniecz commented Jul 9, 2019

I also thought about building regexpes from arrays, but for now went with this.

before: mudwater matched, mud nor water not. After change both mud and water matches as intended.


My work on this pull request was sponsored by a NGI Zero Discovery grant

@goldfndr
Copy link
Contributor

goldfndr commented Jul 9, 2019

The parentheses around all but area=yes look unnecessary, is there a strategy or aesthetic value?

@westnordost
Copy link
Member

Oh, thanks!

@westnordost westnordost merged commit 9e23a58 into streetcomplete:master Jul 10, 2019
@matkoniecz
Copy link
Member Author

The parentheses around all but area=yes look unnecessary, is there a strategy or aesthetic value?

I admit that I am not sure why it is there, but I worry that changing it will break something subtle (and this is not some code that is often changed, so I prefer to do other refactoring).

@matkoniecz matkoniecz deleted the futureproofing branch July 10, 2019 17:13
@matkoniecz matkoniecz mentioned this pull request Jul 25, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants