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.

Sign up for GitHub

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

Jump to bottom

po: Update Portuguese translation #332

Merged
merged 2 commits into from
Feb 12, 2023
Merged

po: Update Portuguese translation #332

merged 2 commits into from
Feb 12, 2023

Conversation

hugok79
Copy link
Contributor

@hugok79 hugok79 commented Jan 3, 2023

No description provided.

mjakeman
mjakeman reviewed Feb 12, 2023
View reviewed changes
Copy link
Owner

@mjakeman mjakeman left a comment

Choose a reason for hiding this comment

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

Hi, everything looks good to me.

One comment about punctuation. Could we use unicode quotation marks in the line above? (Does it make sense for Poruguese to use these?)

Thanks for the contribution!

po/pt.po Outdated Show resolved Hide resolved
@mjakeman
Copy link
Owner

Great, thanks for the contribution!

@mjakeman mjakeman merged commit 9ff6292 into mjakeman:master Feb 12, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@mjakeman mjakeman mjakeman left review comments

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@hugok79 @mjakeman