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 PT-BR automatic translation softwarereview_reviewer.pt.Rmd #818

Open
wants to merge 6 commits into
base: main
Choose a base branch
from

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

…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]>
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

softwarereview_reviewer.pt.Rmd Outdated Show resolved Hide resolved
softwarereview_reviewer.pt.Rmd Outdated Show resolved Hide resolved
@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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Development

Successfully merging this pull request may close these issues.

3 participants