Skip to content

docs: add missing blank line - #16399

Merged
kgryte merged 1 commit into
stdlib-js:developfrom
harsh-thakkar7:chore/5230-exponential-quantile-review
Oct 31, 2026
Merged

kgryte merged 1 commit into
stdlib-js:developfrom
harsh-thakkar7:chore/5230-exponential-quantile-review

Conversation

@harsh-thakkar7

Copy link
Copy Markdown
Contributor

Resolves #5230.

Description

This pull request:

  • adds the missing blank line between the function summary and the annotated parameters in stats/base/dists/exponential/quantile/src/main.c, as requested in the remaining review comment.

The other three review items tracked by #5230 are already resolved on develop: README grammar and C benchmark declaration ordering were addressed by 5186577, and the native-test opts parameters by ef5a939.

Related Issues

Questions

No.

Other

Validation: the exponential quantile JavaScript tests pass 3/3, git diff --check passes, and the change is comment-only. No behavior changes.

Checklist

AI Assistance

  • Yes

  • No

  • Code generation

  • Test/benchmark generation

  • Documentation (including examples)

  • Research and understanding

Disclosure

This PR was prepared with Qoder AI assistance under the contributor's direction. The review comments and final diff were checked locally.


@stdlib-js/reviewers

@harsh-thakkar7
harsh-thakkar7 requested a review from a team October 31, 2026 05:48
@stdlib-bot stdlib-bot added Statistics Issue or pull request related to statistical functionality. Good First PR A pull request resolving a Good First Issue. Needs Review A pull request which needs code review. First-time Contributor A pull request from a contributor who has never previously committed to the project repository. labels Oct 31, 2026
@kgryte kgryte added difficulty: 1 Low degree of difficulty. Should be straightforward to implement and/or resolve. review: 5 and removed Needs Review A pull request which needs code review. labels Oct 31, 2026
@stdlib-bot

Copy link
Copy Markdown
Contributor

Coverage Report

Package Statements Branches Functions Lines
stats/base/dists/exponential/quantile $\\color{green}283/283$
$\\color{green}+100.00\\%$
$\\color{green}23/23$
$\\color{green}+100.00\\%$
$\\color{green}4/4$
$\\color{green}+100.00\\%$
$\\color{green}283/283$
$\\color{green}+100.00\\%$

The above coverage report was generated for the changes in this PR.

@kgryte kgryte left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@kgryte kgryte changed the title docs: add missing blank line in exponential quantile C doc docs: add missing blank line Oct 31, 2026
@kgryte
kgryte merged commit d213b8d into stdlib-js:develop Oct 31, 2026
113 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

difficulty: 1 Low degree of difficulty. Should be straightforward to implement and/or resolve. First-time Contributor A pull request from a contributor who has never previously committed to the project repository. Good First PR A pull request resolving a Good First Issue. review: 5 Statistics Issue or pull request related to statistical functionality.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Address commit comments (commit 8171728)

3 participants