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 5 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://raw.githubusercontent.com/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.

- O LEIAME do seu pacote apresentará um selo de revisão por pares com um link para o tópico de revisão do software.
- Seu pacote receberá um selo [`docs.ropensci.org` site de documentos](#rodocsci) que você pode vincular a partir da DESCRIÇÃO.
- O repositório do seu pacote será transferido para a organização rOpenSci.
- Se os revisores [concordarem em ser listados na DESCRIÇÃO](#authorship) seus metadados mencionarão a revisão.
Copy link
Contributor

Choose a reason for hiding this comment

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

The phrase is not 100% clear to me.
In "their metadata will mention the review.", "their metadata" is referring to the package metadata?
If so, we can make if more explicit:

Suggested change
- Se os revisores [concordarem em ser listados na DESCRIÇÃO](#authorship) seus metadados mencionarão a revisão.
- Se os(as) revisores(as) [concordarem em ser listados(as) no DESCRIPTION](#authorship), os metadados do pacote mencionarão a revisão.

- O repositório do seu pacote será transferido para a organização rOpenSci.
- Se os revisores [concordarem em ser listados na DESCRIÇÃO](#authorship) seus metadados mencionarão a revisão.

## Editores e revisores {#editors-and-reviewers}
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 e revisores {#editors-and-reviewers}
## Editores(as) e revisores(as) {#editors-and-reviewers}
O processo de revisão por pares de software da rOpenSci é conduzido pela nossa equipe dedicada de editores(as) e revisore(as). Informações sobre a equipe atual e o andamento da revisão por pares de software podem ser consultadas em nosso [painel interativo](https://ropensci-review-tools.github.io/dashboard/).
```{r}
#| echo: false
#| results: 'asis'
#| warning: false
if (nzchar(Sys.getenv("AIRTABLE_API_KEY"))) {
source(file.path("scripts", "airtable-get-data.R"), local = knitr::knit_global())
}
```
### Editoria-Chefe
```{r}
#| echo: false
#| results: 'asis'
if (nzchar(Sys.getenv("AIRTABLE_API_KEY"))) {
source(file.path("scripts", "airtable-get-eic.R"), local = knitr::knit_global())
} else {
cat("_No Airtable data available._")
}
```

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._")
}
```

Comment on lines +60 to +84
### Revisores {#reviewers}

Somos gratos às seguintes pessoas que ofereceram seu tempo e conhecimento para revisar os pacotes enviados à rOpenSci.

```{r}
#| echo: false
source(file.path("scripts", "airtable-access.R"), local = knitr::knit_global())
```

```{r}
#| echo: false
#| results: 'asis'
#| eval: !expr has_airtable_access()
source(file.path("scripts", "airtable-get-reviewers.R"), local = knitr::knit_global())
```

Também somos gratos às seguintes pessoas que atuaram como editores convidados.

```{r}
#| echo: false
#| results: 'asis'
#| eval: !expr has_airtable_access()
source(file.path("scripts", "airtable-get-editors.R"), local = knitr::knit_global())

```
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
### Revisores {#reviewers}
Somos gratos às seguintes pessoas que ofereceram seu tempo e conhecimento para revisar os pacotes enviados à rOpenSci.
```{r}
#| echo: false
source(file.path("scripts", "airtable-access.R"), local = knitr::knit_global())
```
```{r}
#| echo: false
#| results: 'asis'
#| eval: !expr has_airtable_access()
source(file.path("scripts", "airtable-get-reviewers.R"), local = knitr::knit_global())
```
Também somos gratos às seguintes pessoas que atuaram como editores convidados.
```{r}
#| echo: false
#| results: 'asis'
#| eval: !expr has_airtable_access()
source(file.path("scripts", "airtable-get-editors.R"), local = knitr::knit_global())
```
### Revisores(as) e editores(as) passados(as)
Agradecemos as seguintes pessoas que dedicaram seu tempo e conhecimento para revisar pacotes submetidos à rOpenSci.
```{r}
#| echo: false
#| results: 'asis'
if (nzchar(Sys.getenv("AIRTABLE_API_KEY"))) {
source(file.path("scripts", "airtable-get-reviewers.R"), local = knitr::knit_global())
} else {
cat("_No Airtable data available._")
}
```
Também agradecemos aos seguintes editores(as) que atuaram anteriormente.
```{r editors_past, echo=FALSE, results='asis'}
if (nzchar(Sys.getenv("AIRTABLE_API_KEY"))) {
cat(gen_ed_out(editors_past), sep = "")
} else {
cat("_No Airtable data available._")
}
```
E aos(as) seguintes que atuaram como editores(as) convidados(as).
```{r guest-editors-out, echo=FALSE, results='asis'}
if (nzchar(Sys.getenv("AIRTABLE_API_KEY"))) {
cat(gen_ed_out(guest_editors), sep = "")
} 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
Co-authored-by: Beatriz Milz <42153618+beatrizmilz@users.noreply.github.com>
Co-authored-by: Beatriz Milz <42153618+beatrizmilz@users.noreply.github.com>
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