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

add locale vi translation #204

Closed
wants to merge 2 commits into from
Closed

Conversation

nhphuc412
Copy link
Contributor

No description provided.

"MIME-Version: 1.0\n"
"Content-Type: text/plain; charset=UTF-8\n"
"Content-Transfer-Encoding: 8bit\n"
"Language: en\n"
Copy link
Member

Choose a reason for hiding this comment

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

@nhphuc412 I believe this is incorrect. This isn't an "en" (English) translation file. If you can update this value to the proper locale entry for the included Vietnamese translations and update this commit we can get this merged in and a new release pushed out.

Thanks.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@cpjolicoeur I updated value for vietnamese translations

@cpjolicoeur
Copy link
Member

@nhphuc412 Thanks so much for the contribution!

Just one small update I believe is needed and then we can merge this in.

@cpjolicoeur
Copy link
Member

@nhphuc412 Thanks so much for the work on this.

Torch v3.6.0 was released today and includes your VI locale.

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.

2 participants