-
Notifications
You must be signed in to change notification settings - Fork 106
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
ADD: improve russian locale #118
Conversation
@warmonger1 can you check on that failing test that Travis is showing? |
Fixed |
@@ -0,0 +1,27 @@ | |||
# -*- encoding: utf-8 -*- |
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.
Shouldn't this file be "en_AU.py" to follow naming convention?
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.
I got name files from self.localeID
variable.
I think for pdtLocale_au
this variable should be equal en_AU
, but compatibility....
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.
ah - that's a mistake then - the localeID should have been set to en_AU
we will have to fix that and then bump the major version number IMO
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.
Agreed. Bump it.
Hold on... Still some problems. Please check my comment @warmonger1 |
@philiptzou sorry - I eager-clicked the merge button! I've fixed the spelling of the en_AU.py file. |
No description provided.