Skip to content

docs: fix typos in hop_index.Rd and slide_index2.Rd#230

Closed
LeonidasZhak wants to merge 2 commits into
r-lib:mainfrom
LeonidasZhak:docs/fix-typos-hop-index-slide-index2
Closed

docs: fix typos in hop_index.Rd and slide_index2.Rd#230
LeonidasZhak wants to merge 2 commits into
r-lib:mainfrom
LeonidasZhak:docs/fix-typos-hop-index-slide-index2

Conversation

@LeonidasZhak

Copy link
Copy Markdown

Summary

Fixes two spelling errors in slider documentation:

  1. hop_index.Rd: "Non-existant" → "Non-existent" (2 occurrences in example comments)
  2. slide_index2.Rd: "multiples inputs" → "multiple inputs" in the \title{}

Changes

File Change
R/hop-index.R Fix "Non-existant" → "Non-existent" in 2 roxygen comments
R/slide-index2.R Fix "multiples" → "multiple" in roxygen title
man/hop_index.Rd Regenerated from roxygen
man/slide_index2.Rd Regenerated from roxygen

Validation

  • tools::checkRd("man/hop_index.Rd") — passes (no output = no errors)
  • tools::checkRd("man/slide_index2.Rd") — passes (no output = no errors)
  • All 1229 functional tests pass (5 pre-existing snapshot failures from vctrs version mismatch, unrelated)
  • 4 files changed, 6 lines modified

Add a grouped rolling mean example showing the Stata-to-R migration
pattern for panel data. This is the most common use case for users
migrating from Stata's rangestat/bysort + rolling operations.

The example shows:
- Panel data with two firms and irregular dates
- 3-day rolling mean within each firm using slide_index_mean()
- Date-aware windows that respect gaps in the series

Related: r-lib#193
- 'Non-existant' -> 'Non-existent' in hop_index examples (2 occurrences)
- 'multiples inputs' -> 'multiple inputs' in slide_index2 title
@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