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

Review for Portuguese translation of maintenance_marketing.pt.Rmd #798

Conversation

pedropark99
Copy link
Contributor

This first review adds two changes:

  • Remove a section in the last bulletpoint that is not present in the original file, to be a more precise translation of the original content.
  • Fix some technical terms that got twisted in the translation.

@pedropark99
Copy link
Contributor Author

This PR is ready for the 2nd review.

@pedropark99
Copy link
Contributor Author

Thank you very much @IldySilva !!!

@rffontenelle
Copy link
Contributor

I see this in "second review" column in the project view, but it has 2 reviews. Should it go to "To be merged"?

@pedropark99
Copy link
Contributor Author

@yabellini I agree with @rffontenelle . I also understand that this should be moved to "To be merged".

@yabellini
Copy link
Member

@yabellini I agree with @rffontenelle . I also understand that this should be moved to "To be merged".

Thank you! Moved.

@yabellini yabellini merged commit 4553565 into ropensci:maintenance_marketing.pt.Rmd-pt-auto Nov 18, 2024
yabellini added a commit that referenced this pull request Nov 18, 2024
* Add PT-BR automatic translation maintenance_marketing.pt.Rmd

* fix alias

* fix yaml

* Review for Portuguese translation of `maintenance_marketing.pt.Rmd` (#798)

---------

Co-authored-by: Pedro Faria <69123925+pedropark99@users.noreply.github.com>
Co-authored-by: Yanina Bellini Saibene <yabellini@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants