-
-
Notifications
You must be signed in to change notification settings - Fork 1.4k
Added Danish translation #12333
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: main
Are you sure you want to change the base?
Added Danish translation #12333
Conversation
|
ai, please recheck |
|
ai? this isn't AI, what's the problem? |
|
I am working on an ai bot to check translations but it worked on my private repo but is throwing a tantrum here dw |
|
ah, okay no worries |
|
ai, please recheck |
|
AI translation check result: Translation check not ok
|
|
finally, works :) |
|
Nice! Do you want me to correct it following what it's saying? Because i slightly disagree with some of the suggestions |
|
if you disagree it's fine, but I see typos so those should probably be fixed. the purpose of the ci job is twofold:
|
|
Makes sense, i'll definitely correct the spelling mistake. |
that's why I'm saying AI is a guideline and not a hard requirement xD |
vaxerski
left a comment
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.
thanks!
|
You're welcome! |
|
clang-format needed |
|
i'm hoping that did it, i don't code C++ myself |
|
just run |
vaxerski
left a comment
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.
thanks!
Describe your PR, what does it fix/add?
This PR adds Danish language translations
Is there anything you want to mention? (unchecked code, possible bugs, found problems, breaking compatibility, etc.)
None
Is it ready for merging, or does it need work?
Yes. The translations are finished.