Skip to content

docs: add missing @return tags to exported functions - #846

Closed
LeonidasZhak wants to merge 1 commit into
r-lib:mainfrom
LeonidasZhak:add-return-tags
Closed

docs: add missing @return tags to exported functions#846
LeonidasZhak wants to merge 1 commit into
r-lib:mainfrom
LeonidasZhak:add-return-tags

Conversation

@LeonidasZhak

Copy link
Copy Markdown

Add @return documentation tags to exported functions whose generated Rd files lacked \value sections. R CMD check --as-cran notes missing \value for exported functions; this is also a CRAN submission requirement.

Functions documented

Function Returns
ctl_new_pillar_list() A list of pillar objects
new_pillar_component() / pillar_component() A pillar_component object
new_pillar_shaft() A pillar shaft object of class c(class, "pillar_shaft")
pillar_shaft() A pillar shaft object
new_pillar_title() A pillar_title object
new_pillar_type() A pillar_type object
format_type_sum() A styled character string
scale_x_num() / scale_y_num() A ggplot2 scale object

Validation

  • R CMD build --no-build-vignettes: succeeds
  • R CMD check --no-manual --no-vignettes --no-tests: 2 WARNINGs (pre-existing vignette issues, unrelated)
  • Rd regeneration: clean (roxygen2::roxygenise() succeeded)
  • No new issues introduced by this change

Diff

36 files changed, 109 insertions, 2 deletions

Add @return documentation tags to exported functions whose generated
Rd files lacked \value sections. R CMD check --as-cran notes missing
\value for exported functions; this is also a CRAN submission requirement.

Functions documented:
- ctl_new_pillar_list(): list of pillar objects
- new_pillar_component()/pillar_component(): pillar_component object
- new_pillar_shaft(): pillar shaft object
- pillar_shaft(): pillar shaft object
- new_pillar_title(): pillar_title object
- new_pillar_type(): pillar_type object
- format_type_sum(): styled character string
- scale_x_num()/scale_y_num(): ggplot2 scale object

Also regenerated all Rd files to include pre-existing @return tags
that were present in the R source but not reflected in man pages.
@LeonidasZhak

Copy link
Copy Markdown
Author

Withdrawing this small automated PR while I consolidate an oversized batch of contributions and reduce maintainer review burden. Sorry for the noise, and thank you for maintaining the project.

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.

1 participant