Skip to content

Correct commented argument name#6407

Merged
ben-schwen merged 2 commits intomasterfrom
lazyarg-name
Aug 28, 2024
Merged

Correct commented argument name#6407
ben-schwen merged 2 commits intomasterfrom
lazyarg-name

Conversation

@MichaelChirico
Copy link
Copy Markdown
Member

@MichaelChirico MichaelChirico commented Aug 27, 2024

SEXP bmerge(SEXP idt, SEXP xdt, SEXP icolsArg, SEXP xcolsArg, SEXP xoArg, SEXP rollarg, SEXP rollendsArg, SEXP nomatchArg, SEXP multArg, SEXP opArg, SEXP nqgrpArg, SEXP nqmaxgrpArg) {

@github-actions
Copy link
Copy Markdown

github-actions bot commented Aug 27, 2024

Comparison Plot

Generated via commit caec41a

Download link for the artifact containing the test results: ↓ atime-results.zip

Time taken to finish the standard R installation steps: 11 minutes and 54 seconds

Time taken to run atime::atime_pkg on the tests: 5 minutes and 48 seconds

@ben-schwen ben-schwen merged commit 2b7d3da into master Aug 28, 2024
@ben-schwen ben-schwen deleted the lazyarg-name branch August 28, 2024 08:43
MichaelChirico added a commit that referenced this pull request Aug 30, 2024
* Correct commented argument name

* also mismatched bmerge() declaration
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.

2 participants