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

use danish lang code, not country. fixes #6908 #6909

Merged
merged 1 commit into from
Jul 12, 2022
Merged

use danish lang code, not country. fixes #6908 #6909

merged 1 commit into from
Jul 12, 2022

Conversation

eshellman
Copy link
Collaborator

@eshellman eshellman commented Jul 11, 2022

What does this PR do?

changes suffix for danish to use language cod da in stead of country code for denmark dk

Why is this valuable (or not)?

We try to be consistent about using language codes. fixes #6908

Checklist:

Follow-up

  • Check the status of GitHub Actions and resolve any reported warnings!

@eshellman eshellman changed the title use danish lang code, not country use danish lang code, not country. fixes #6908 Jul 11, 2022
@davorpa davorpa added 🗣️ locale:da Resources addressing "Danish / dansk" language 🗣️ translations Issues or PRs addresing translations labels Jul 11, 2022
@davorpa davorpa self-requested a review July 11, 2022 17:14
@davorpa davorpa added the 🚀 ready to merge LGTM. Waiting for final approval label Jul 11, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🚀 ready to merge LGTM. Waiting for final approval 🗣️ locale:da Resources addressing "Danish / dansk" language 🗣️ translations Issues or PRs addresing translations
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Danish full ICU code is da-dk not dk
2 participants