Skip to content

Add PT-BR automatic translation softwarereview_intro.pt.Rmd #822

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

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

Conversation

yabellini
Copy link
Member

No description provided.

@beatrizmilz
Copy link
Contributor

beatrizmilz commented Nov 18, 2024

Vou fazer a primeira revisão!

@rffontenelle
Copy link
Contributor

@yabellini considering my first review, can you please mark need second review?

@yabellini
Copy link
Member Author

Thanks @rffontenelle !! Status changed 😄

@yabellini
Copy link
Member Author

@rffontenelle where we can see your first review?

@rffontenelle
Copy link
Contributor

Screenshot_20241118-114147.jpg

Weird. I'll check that later in my computer

@rffontenelle
Copy link
Contributor

@beatrizmilz I actually barely started and stopped. So please feel free to proceed reviewing this.

@yabellini So, we are actually in need of the first review. You may want to undo the labels changes.

Sorry for the noise!

@yabellini
Copy link
Member Author

No problem at all!
I will update and thanks for all the work you are doing for the translation

@yabellini
Copy link
Member Author

The first review is happening here: https://github.com/ropensci/dev_guide/pull/852/files by @beatrizmilz
She will let us know when it is ready for a second review.
Work started in the Traslatón.

@brunomioto
Copy link

If it's a need, I can do the second review.

@yabellini
Copy link
Member Author

Hi @brunomioto ! Thanks for your offer. 🙏
Bea is still working on this PR; we can let you know when it is ready. In the meantime, if you want to do a second review, this PR needs one: #824

@yabellini yabellini requested a review from a team as a code owner April 18, 2025 20:51
@yabellini yabellini requested review from beatrizmilz and removed request for a team April 18, 2025 20:51
Copy link
Contributor

@beatrizmilz beatrizmilz left a comment

Choose a reason for hiding this comment

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

Hi @yabellini ! I have migrated all the comments/suggestions from #852 to this PR. Is that ok? I think it will be easier for us to follow up the review here.

And since this PR is older, I don't remember why I did create a new (#852) in the first place.

I will copy the message I have left in the other PR:


Hi! I have finished the first review of this chapter.

I have based the review of the translation for this document: https://github.com/raw/ropensci/dev_guide/refs/heads/main/softwarereview_intro.Rmd - there are some things that were diferent (probably because this branch was from some months ago) and I have used that file to check if anything needed to be updated.

I have left some comments in the translation so we can discuss.

Comment on lines 52 to 58
### Editores associados {#associateditors}

O processo de revisão por pares de software da rOpenSci é conduzido por:

```{r}
#| child: "associate-editors.md"
```
Copy link
Contributor

Choose a reason for hiding this comment

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

This part is quite different because the file in English has been updated since the begining of the translation process:
https://github.com/ropensci/dev_guide/blob/main/softwarereview_intro.Rmd

Suggested change
### Editores associados {#associateditors}
O processo de revisão por pares de software da rOpenSci é conduzido por:
```{r}
#| child: "associate-editors.md"
```
### Equipe editorial
Nossa equipe atual de editores(as) para revisão por pares de software inclui:
```{r}
#| echo: false
#| results: 'asis'
if (nzchar(Sys.getenv("AIRTABLE_API_KEY"))) {
source(file.path("scripts", "airtable-get-editors.R"), local = knitr::knit_global())
} else {
cat("_No Airtable data available._")
}
```

@yabellini yabellini moved this from Todo - First Review to Todo - Second Review in Tradução Português- Guia do programador Apr 19, 2025
@yabellini
Copy link
Member Author

Thank you @beatrizmilz ! This is ready for your review @fblpalmeira .

yabellini and others added 2 commits April 19, 2025 13:20
@maelle maelle requested a review from fblpalmeira April 24, 2025 10:41
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.

4 participants