Skip to content
This repository has been archived by the owner on Jun 28, 2021. It is now read-only.

Sort translation languages list in alphabetical order #138

Merged
merged 3 commits into from
Oct 15, 2015

Conversation

reshadn
Copy link
Contributor

@reshadn reshadn commented Oct 15, 2015

Fixes #79
screen shot 2015-10-15 at 1 42 05 am
screen shot 2015-10-15 at 1 43 12 am
screen shot 2015-10-15 at 1 43 25 am
screen shot 2015-10-15 at 1 43 37 am

@mmahalwy
Copy link
Contributor

lgtm!

@mmahalwy
Copy link
Contributor

Thanks so much!

@reshadn
Copy link
Contributor Author

reshadn commented Oct 15, 2015

Yeah this was fun.

@mmahalwy Sad that the CI builds failed. It says "No environment variables set". I'm assuming that's a configuration issue and not a tests failing issue?

@reshadn
Copy link
Contributor Author

reshadn commented Oct 15, 2015

@mmahalwy After your lgtm do I merge my own PR? Any other processes to follow? Thanks.

@mmahalwy
Copy link
Contributor

@reshadn it's a problem on my end. your tests are good. And yes, process is to merge your own (for those that have access at least...)

reshadn added a commit that referenced this pull request Oct 15, 2015
Sort translation languages list in alphabetical order
@reshadn reshadn merged commit adf6ec6 into master Oct 15, 2015
@reshadn reshadn deleted the sorting-translations-list branch October 15, 2015 21:23
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants