-
-
Notifications
You must be signed in to change notification settings - Fork 80
update Luxembourg subdivision codes #77
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
base: master
Are you sure you want to change the base?
Conversation
✅ Deploy Preview for biter777countries ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
While I understand the need to update, I feel strange existing constant are nuked
If they were kept with a deprecated comment, and assigned to the new one with the switch
|
Ah didnt realise this was an option, will update |
47c3130 to
b3e1b8c
Compare
|
Hi @ccoVeille, i've updated the PR to put the old codes back in. I couldnt see an obvious way to delegate the old codes to the new ones, so at the moment the list is now the superset of both. Let me know if you want somethng different. |
592980f to
fb055b4
Compare
|
Done, thanks |
fb055b4 to
00b60f7
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM 👍
Let's wait for a maintainer feedback now
00b60f7 to
77fb01d
Compare
|
spotted a duplicate in the case statement, fixed |
as per https://en.wikipedia.org/wiki/ISO_3166-2:LU