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: sk_SK street names #675

Closed
wants to merge 2 commits into from

Conversation

devicebusy
Copy link

@devicebusy devicebusy commented Aug 1, 2023

What is the reason for this PR?

Fixed an issue with sk_SK street names generation (resolve #672)

Author's checklist

Summary of changes

Added a list of real SK streets, fixed street name format

Review checklist

  • All checks have passed
  • Changes are approved by maintainer

@pimjansen
Copy link

Please review the contribution guide (see pr template)

@pimjansen pimjansen added invalid This doesn't seem right needs work labels Aug 12, 2023
@localheinz localheinz self-assigned this Sep 3, 2023
@localheinz localheinz added bug Something isn't working and removed invalid This doesn't seem right needs work labels Sep 3, 2023
@localheinz
Copy link
Member

@pimjansen

This looks good to me - do you have any objections?

@pimjansen
Copy link

@pimjansen

This looks good to me - do you have any objections?

The code itself looks ok but i'd rather see the locale contents covered by a test

@localheinz localheinz changed the base branch from 2.0 to 1.23 September 9, 2023 07:25
@localheinz localheinz added this to the 1.23.1 milestone Sep 9, 2023
@stale
Copy link

stale bot commented Oct 15, 2023

This issue has been automatically marked as stale because it has not had recent activity. It will be closed after 1 week if no further activity occurs. Thank you for your contributions.

@bram-pkg bram-pkg closed this Nov 14, 2023
@MilosM33
Copy link

@pimjansen Hello, I'd like to reopen and finalize this Pull Request. What specific test would you like to see?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working lifecycle/stale
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Incorect output value for sk_SK location from streetName
5 participants