-
Notifications
You must be signed in to change notification settings - Fork 63
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
base: main
Are you sure you want to change the base?
Conversation
Vou fazer a primeira revisão! |
@yabellini considering my first review, can you please mark need second review? |
Thanks @rffontenelle !! Status changed 😄 |
@rffontenelle where we can see your first review? |
@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! |
No problem at all! |
The first review is happening here: https://github.com/ropensci/dev_guide/pull/852/files by @beatrizmilz |
If it's a need, I can do the second review. |
Hi @brunomioto ! Thanks for your offer. 🙏 |
There was a problem hiding this 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. |
There was a problem hiding this comment.
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:
- 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} |
There was a problem hiding this comment.
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
## 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._") | |
} | |
``` | |
### Editores associados {#associateditors} | ||
|
||
O processo de revisão por pares de software da rOpenSci é conduzido por: | ||
|
||
```{r} | ||
#| child: "associate-editors.md" | ||
``` |
There was a problem hiding this comment.
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
### 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._") | |
} | |
``` |
### 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()) | ||
|
||
``` |
There was a problem hiding this comment.
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
### 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._") | |
} | |
``` |
Thank you @beatrizmilz ! This is ready for your review @fblpalmeira . |
Co-authored-by: Beatriz Milz <42153618+beatrizmilz@users.noreply.github.com>
Co-authored-by: Beatriz Milz <42153618+beatrizmilz@users.noreply.github.com>
No description provided.