Skip to content

Add PT-BR automatic translation softwarereview_reviewer.pt.Rmd #818

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

Merged
merged 9 commits into from
Mar 27, 2025

Conversation

yabellini
Copy link
Member

No description provided.

@pedropark99
Copy link
Contributor

First review done at #851

@pedropark99
Copy link
Contributor

@yabellini this one is ready to be moved to "Second review"

@yabellini
Copy link
Member Author

Done! Thanks @pedropark99 !

@samuel-carleial
Copy link
Contributor

I am working on this task. I hope my commits (PR) will work out :-)

@samuel-carleial
Copy link
Contributor

samuel-carleial commented Nov 29, 2024

It did not work out as it was in a wrong branch. See my new PR (hopefully) on the correct branch: #854

Copy link
Member Author

@yabellini yabellini left a comment

Choose a reason for hiding this comment

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

Removing YAML

@yabellini
Copy link
Member Author

@pedropark99 I don't know if you saw this PR by @samuel-carleial L: https://github.com/ropensci/dev_guide/pull/854/files

It has some other relevant suggestions. Before I merge to main, could you please check this and let me know how to proceed? Thank you!

@pedropark99
Copy link
Contributor

Oh I did not see this other PR, thank you! I will take a look at it.

Copy link
Member Author

@yabellini yabellini left a comment

Choose a reason for hiding this comment

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

This are suggestions made by @samuel-carleial in this PR: https://github.com/ropensci/dev_guide/pull/854/files
Some of the suggestions were already in the PR made by other reviewers. I migrate here those that weren't but help with the consistency of the translation. Thanks for this review!

@yabellini
Copy link
Member Author

@maelle, this PR is also ready to be merged now. I am letting you do the merge since you know the changes we are making.

yabellini and others added 9 commits March 27, 2025 13:29
…Rmd` (#851)

* Add review

* Add review

* Review until line 44

* Finish review

* Update softwarereview_reviewer.pt.Rmd

* Update softwarereview_reviewer.pt.Rmd

Co-authored-by: Ariana <[email protected]>

* Update softwarereview_reviewer.pt.Rmd

Co-authored-by: Ariana <[email protected]>

* Update softwarereview_reviewer.pt.Rmd

Co-authored-by: Ariana <[email protected]>

* Update softwarereview_reviewer.pt.Rmd

Co-authored-by: Ariana <[email protected]>

* Update softwarereview_reviewer.pt.Rmd

Co-authored-by: Ariana <[email protected]>

* Update softwarereview_reviewer.pt.Rmd

Co-authored-by: Ariana <[email protected]>

* Update softwarereview_reviewer.pt.Rmd

Co-authored-by: Ariana <[email protected]>

* Update softwarereview_reviewer.pt.Rmd

Co-authored-by: Ariana <[email protected]>

* Update softwarereview_reviewer.pt.Rmd

Co-authored-by: Ariana <[email protected]>

* Update softwarereview_reviewer.pt.Rmd

Co-authored-by: Ariana <[email protected]>

* Update softwarereview_reviewer.pt.Rmd

Co-authored-by: Ariana <[email protected]>

* Update softwarereview_reviewer.pt.Rmd

Co-authored-by: Ariana <[email protected]>

* Update softwarereview_reviewer.pt.Rmd

Co-authored-by: Ariana <[email protected]>

* Update softwarereview_reviewer.pt.Rmd

Co-authored-by: Ariana <[email protected]>

* Update softwarereview_reviewer.pt.Rmd

Co-authored-by: Ariana <[email protected]>

* Update softwarereview_reviewer.pt.Rmd

Co-authored-by: Ariana <[email protected]>

* Update softwarereview_reviewer.pt.Rmd

Co-authored-by: Ariana <[email protected]>

* Update softwarereview_reviewer.pt.Rmd

Co-authored-by: Ariana <[email protected]>

* Update softwarereview_reviewer.pt.Rmd

Co-authored-by: Ariana <[email protected]>

* Update softwarereview_reviewer.pt.Rmd

Co-authored-by: Ariana <[email protected]>

---------

Co-authored-by: Ariana <[email protected]>
@maelle maelle force-pushed the softwarereview_reviewer.pt.Rmd-pt-auto branch from 995948b to abab8d7 Compare March 27, 2025 12:29
@maelle
Copy link
Member

maelle commented Mar 27, 2025

Thanks all!

@maelle maelle merged commit 5fdf24f into main Mar 27, 2025
1 check passed
@github-project-automation github-project-automation bot moved this from To be merged to Todo - Second Review in Tradução Português- Guia do programador Mar 27, 2025
@yabellini yabellini moved this from Todo - Second Review to To be merged in Tradução Português- Guia do programador Mar 27, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

Successfully merging this pull request may close these issues.

4 participants