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

[FEAT] Include waterway/stream in tags-to-keep.json #201

Merged
merged 1 commit into from
May 5, 2023

Conversation

treee111
Copy link
Owner

@treee111 treee111 commented May 5, 2023

This PR…

How to test

  1. ...
  2. ...

Pull Request Checklist

@treee111 treee111 changed the base branch from develop to release-v4.0.0 May 5, 2023 19:10
@treee111 treee111 changed the title Include waterway stream [FEAT] Include waterway/stream in tags-to-keep.json May 5, 2023
@treee111 treee111 changed the base branch from release-v4.0.0 to develop May 5, 2023 19:14
@treee111 treee111 force-pushed the include-waterway-stream branch 2 times, most recently from a2ca8eb to 4c6a04a Compare May 5, 2023 19:15
@treee111 treee111 force-pushed the include-waterway-stream branch from 4c6a04a to 362ae01 Compare May 5, 2023 19:16
@treee111 treee111 merged commit f6a4b99 into develop May 5, 2023
@treee111 treee111 deleted the include-waterway-stream branch May 5, 2023 19:17
treee111 added a commit that referenced this pull request Nov 2, 2023
treee111 added a commit that referenced this pull request Nov 2, 2023
- due to changes in included OSM elements from #206 b8f4d16 and #201 f6a4b99
treee111 added a commit that referenced this pull request Nov 3, 2023
- due to changes in included OSM elements from #206 b8f4d16 and #201 f6a4b99
treee111 added a commit that referenced this pull request Nov 3, 2023
- due to changes in included OSM elements from #206 b8f4d16 and #201 f6a4b99
treee111 added a commit that referenced this pull request Nov 3, 2023
- plus adding 100/137 as this was not inclued before in Windows, but in macOS
- due to changes in included OSM elements from #206 b8f4d16 and #201 f6a4b99
treee111 added a commit that referenced this pull request Nov 3, 2023
- due to changes in included OSM elements from #206 b8f4d16 and #201 f6a4b99
treee111 added a commit that referenced this pull request Nov 3, 2023
- plus adding 100/137 as this was not inclued before in Windows, but in macOS
- due to changes in included OSM elements from #206 b8f4d16 and #201 f6a4b99
treee111 added a commit that referenced this pull request Nov 4, 2023
* update static unittest files macOS

- due to changes in included OSM elements from #206 b8f4d16 and #201 f6a4b99

* Adjust test_constants to match the new tags added to tags-to-keep.json

* update static unittest files Windows

- plus adding 100/137 as this was not inclued before in Windows, but in macOS
- due to changes in included OSM elements from #206 b8f4d16 and #201 f6a4b99

* integrate unittests into CONTRIBUTING.md

- incl. refactoing the file & link to land-poligons
- remove not needed headlines

---------

Co-authored-by: Alf Høgemark <[email protected]>
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.

waterway=stream is missing in tags-to-keep.json
1 participant