Skip to content

docs: fix minmax_run description and subtracted typo#114

Closed
LeonidasZhak wants to merge 1 commit into
gogonzo:mainfrom
LeonidasZhak:docs/fix-minmax-run-description
Closed

docs: fix minmax_run description and subtracted typo#114
LeonidasZhak wants to merge 1 commit into
gogonzo:mainfrom
LeonidasZhak:docs/fix-minmax-run-description

Conversation

@LeonidasZhak

Copy link
Copy Markdown
Contributor

Summary

Fixes two documentation issues in runner package:

  1. minmax_run.Rd: Description incorrectly referenced min_run instead of minmax_run, and mentioned "specified k window size" despite minmax_run having no k parameter.

  2. dot-reformat_k.Rd (from R/utils.R): Typo "substracted" → "subtracted".

Changes

File Change
src/minmax_run.cpp Fix roxygen comment: min_runminmax_run, remove k reference
R/RcppExports.R Auto-regenerated from C++ source
R/utils.R Fix "substracted" → "subtracted" typo
man/minmax_run.Rd Regenerated
man/dot-reformat_k.Rd Regenerated

Validation

  • tools::checkRd("man/minmax_run.Rd") — passes
  • tools::checkRd("man/dot-reformat_k.Rd") — passes
  • R CMD build . — succeeds
  • R CMD check runner_0.5.0.tar.gz --no-manual — Status: 1 WARNING (pre-existing dplyr undeclared), 1 NOTE (pre-existing .claude hidden dir), both unrelated to this change
  • All examples run correctly

Related

- minmax_run.Rd: fix description from 'min_run' to 'minmax_run',
  remove incorrect 'k window size' reference (function has no k param)
- utils.R/.reformat_k: fix 'substracted' -> 'subtracted' typo

Changes in roxygen source (src/minmax_run.cpp, R/utils.R),
Rd files regenerated with roxygen2.
@LeonidasZhak

Copy link
Copy Markdown
Contributor 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