Skip to content

fix typo#1000

Merged
maelle merged 3 commits intoropensci:mainfrom
Pakillo:patch-1
Apr 10, 2026
Merged

fix typo#1000
maelle merged 3 commits intoropensci:mainfrom
Pakillo:patch-1

Conversation

@Pakillo
Copy link
Copy Markdown
Member

@Pakillo Pakillo commented Apr 9, 2026

I believe there is a missing @ symbol in #' noRd


Checklist for dev guide maintainers, do not delete 😸

  • Review of the content in the initial language.
  • News item.
  • Translation of the content in other languages.
  • Review of the translations.

@mpadge
Copy link
Copy Markdown
Member

mpadge commented Apr 9, 2026

Thanks for catching that @Pakillo! Can you please also update same typo here:

- Añade `#' @keywords internal` para marcar una función como interna sin dejar de generar su documentación. Si no quieres que se genere documentación de la función, utiliza `#' noRd` en su lugar. Consulta la documentación de [`roxygen2` sobre etiquetas para indexación y referencias cruzadas](https://roxygen2.r-lib.org/reference/tags-index-crossref.html) y [etiquetas para documentar funciones](https://roxygen2.r-lib.org/reference/tags-rd.html). Para fines de desarrollo, puede que te interese la etiqueta [paquete experimental devtag](https://github.com/moodymudskipper/devtag) para obtener páginas de manual locales al utilizar `#' @noRd`.

and here:
- Adicionar `#' @keywords internal` para marcar uma função como interna e, ao mesmo tempo, gerar documentação para ela. Se você não quiser que nenhuma documentação de função seja gerada, use `#' noRd`. Consulte [a documentação do `roxygen2` sobre tags para indexação e referência cruzada](https://roxygen2.r-lib.org/reference/tags-index-crossref.html) e [tags para funções de documentação](https://roxygen2.r-lib.org/reference/tags-rd.html). Para fins de desenvolvimento, você pode se interessar pelo [pacote experimental devtag](https://github.com/moodymudskipper/devtag) para obter páginas de manual locais ao usar o `#' @noRd`.

We also usually insert line breaks at end of sentences, so feel free to break each of those into 4 lines instead of one long one. Thanks 👍

@Pakillo
Copy link
Copy Markdown
Member Author

Pakillo commented Apr 9, 2026

ok, thanks, Mark. Will update Spanish and Portuguese Rmd too

@Pakillo Pakillo requested a review from a team as a code owner April 9, 2026 13:15
@Pakillo Pakillo requested review from yabellini and removed request for a team April 9, 2026 13:15
@Pakillo Pakillo requested a review from a team as a code owner April 9, 2026 13:16
@Pakillo Pakillo requested review from fblpalmeira and removed request for a team April 9, 2026 13:16
@Pakillo
Copy link
Copy Markdown
Member Author

Pakillo commented Apr 9, 2026

Done!

Copy link
Copy Markdown
Member

@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.

Thank you @Pakillo ! This looks good.

@maelle
Copy link
Copy Markdown
Member

maelle commented Apr 10, 2026

Muchas gracias!!

@maelle
Copy link
Copy Markdown
Member

maelle commented Apr 10, 2026

No need for a review for PT since this is code only.

@maelle maelle merged commit 6ab2991 into ropensci:main Apr 10, 2026
1 check passed
Copy link
Copy Markdown
Contributor

@fblpalmeira fblpalmeira left a comment

Choose a reason for hiding this comment

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

No changes required.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants