Skip to content

Conversation

muffinresearch
Copy link
Contributor

@muffinresearch muffinresearch commented May 20, 2016

@coveralls
Copy link

Coverage Status

Coverage remained the same at 100.0% when pulling dccae3b on muffinresearch:create-locales into 682b254 on mozilla:master.

locale = locale.replace('-', '_');
shell.exec(`mkdir -p ${localeDir}/${locale}/LC_MESSAGES/`);
for (potFile of potFiles) {
fileName = `${path.basename(potFile, '.pot')}`;
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The template string isn't needed here.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Heh, yes - thanks good catch!

@mstriemer
Copy link
Contributor

r+wc

@muffinresearch muffinresearch merged commit c60d844 into mozilla:master May 20, 2016
@muffinresearch muffinresearch deleted the create-locales branch May 20, 2016 16:01
@coveralls
Copy link

Coverage Status

Coverage remained the same at 100.0% when pulling b082e37 on muffinresearch:create-locales into 682b254 on mozilla:master.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Setup locale creation
3 participants