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

Update NL locale. #1144

Merged
merged 1 commit into from
Jan 25, 2017
Merged

Update NL locale. #1144

merged 1 commit into from
Jan 25, 2017

Conversation

viroulep
Copy link
Member

Submitted by Ron.

Submitted by Ron.
@jonatanklosko
Copy link
Member

I don't think there's any sense in waiting for someone's approval, as that's just some language stuff. IMO we should always merge such PRs immediately =)

@viroulep viroulep merged commit c9b5331 into thewca:master Jan 25, 2017
@viroulep viroulep deleted the update-nl branch January 25, 2017 14:41
@jfly
Copy link
Contributor

jfly commented Jan 25, 2017

I agree @jonatanklosko, I've updated our policy for merging code to say:

If the PR addresses a critical issue or is just updating a translation, then you can merge it up immediately

@jfly jfly added the i18n label Jan 31, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants