Skip to content

roxygen2 examplesIf tag already on CRAN#1001

Merged
maelle merged 5 commits intoropensci:mainfrom
Pakillo:patch-2
Apr 10, 2026
Merged

roxygen2 examplesIf tag already on CRAN#1001
maelle merged 5 commits intoropensci:mainfrom
Pakillo:patch-2

Conversation

@Pakillo
Copy link
Copy Markdown
Member

@Pakillo Pakillo commented Apr 9, 2026


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.

Removed reference to the roxygen2 development version for the @exampleIf tag.
@mpadge
Copy link
Copy Markdown
Member

mpadge commented Apr 9, 2026

Cool, thanks @Pakillo , and same comment as #1000, if you could also check .es and .pt versions, and update there if necessary.

And that made me realise you'd got us to the milestone of #1000!!! Congratulations 🎉 ❗ @maelle We should probably add a tiny news item; a pretty cool acheivement 😄

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

Pakillo commented Apr 9, 2026

Done! I've also changed exampleIf to examplesIf, as I understand that's the proper roxygen tag? cf. https://roxygen2.r-lib.org/reference/tags-rd.html

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! this looks good.

@maelle
Copy link
Copy Markdown
Member

maelle commented Apr 10, 2026

Thanks a ton @Pakillo!

I added a news item @mpadge!

@maelle maelle merged commit 2e7fca2 into ropensci:main Apr 10, 2026
1 check failed
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.

4 participants