Skip to content

chore: propagate recent doctest and lint fixes to sibling packages#11585

Merged
kgryte merged 5 commits intodevelopfrom
philipp/fix-propagation-2026-04-19
Apr 19, 2026
Merged

chore: propagate recent doctest and lint fixes to sibling packages#11585
kgryte merged 5 commits intodevelopfrom
philipp/fix-propagation-2026-04-19

Conversation

@Planeshifter
Copy link
Copy Markdown
Member

@Planeshifter Planeshifter commented Apr 19, 2026

Description

Propagates four recent develop fixes to sibling packages with the same defect. One commit per source.

blas/ext/base/ndarray/dcusum*/docs/repl.txt — source 638fdf0

Collapse the two-step > {{alias}}( ... ); + > {{alias:@stdlib/ndarray/to-array}}( y ) REPL doctest into the inline <ndarray>[ ... ] form across the five dcusum* siblings (dcusum, dcusumkbn, dcusumkbn2, dcusumors, dcusumpw).

blas/docs/types/index.d.ts — source 97d6f49

Inline the sdot @example scalar return the same way the adjacent ddot entry already is: drop var v = z.get(); + // returns -5.0 and fold the value into // returns <ndarray>[ -5.0 ].

stats/base/ndarray/cumin/lib/main.js — source 0873811

Remove the orphan require( '@stdlib/ndarray/to-array' ) left behind in the @example block (the output is already inlined).

utils/async/{inmap,inmap-right,for-each} + datasets/us-states-capitals-names — source 650bf8a

Replace new Error( 'text: ' + value ) with new Error( format( 'text: %s', value ) ) and add the @stdlib/string/format import.

Questions

No.

Other

No related-issue keywords on purpose — random/iter (49fc241) and napi/create-* (e08ef82, b946af9) had no remaining candidates after their source commits.

Checklist

AI Assistance

  • Yes
  • No

If you answered "yes" above, how did you use AI assistance?

  • Code generation (e.g., when writing an implementation or fixing a bug)
  • Test/benchmark generation
  • Documentation (including examples)
  • Research and understanding

Disclosure

Authored by Claude Code running a fix-propagation routine: enumerated commits merged to develop in the last 24h, extracted generalizable patterns, searched for sibling sites, validated with independent reviewer agents, and applied only the high-signal propagations. Draft until a maintainer audits.


@stdlib-js/reviewers

claude added 4 commits April 19, 2026 13:50
…y/dcusum*`

Propagates fix from 638fdf0 ("docs: improve doctests for ndarray instances in `blas/ext/base/ndarray/gcusumkbn2`") to the five `dcusum*` sibling packages (dcusum, dcusumkbn, dcusumkbn2, dcusumors, dcusumpw), collapsing the two-step `{{alias}}(...);` + `{{alias:@stdlib/ndarray/to-array}}(y)` REPL doctest into the inline `<ndarray>[ ... ]` form.
Propagates fix from 97d6f49 ("docs: improve doctests for ndarray instances in `blas/sdot`") to the `sdot` @example entry in `blas/docs/types/index.d.ts`, collapsing the `// returns <ndarray>` + `var v = z.get();` two-step pattern into the inline `<ndarray>[ -5.0 ]` form used by the adjacent `ddot` entry.
…min`

Propagates fix from 0873811 ("docs: inline ndarray output in doctests and add missing article") to `stats/base/ndarray/cumin/lib/main.js`, removing the orphan `@stdlib/ndarray/to-array` require in the `@example` block since the output is already shown via the inline `<ndarray>[ ... ]` form.
Propagates fix from 650bf8a ("chore: fix JavaScript lint errors") to sibling example files that still build error messages via string concatenation. Replaces `new Error( 'text: ' + value )` with `new Error( format( 'text: %s', value ) )` and adds the `@stdlib/string/format` import across `utils/async/inmap`, `utils/async/inmap-right`, `utils/async/for-each`, and `datasets/us-states-capitals-names` examples.
@stdlib-bot
Copy link
Copy Markdown
Contributor

stdlib-bot commented Apr 19, 2026

Coverage Report

Package Statements Branches Functions Lines
blas $\color{red}98788/122179$
$\color{green}+0.00%$
$\color{red}1754/1971$
$\color{green}+0.00%$
$\color{red}33/997$
$\color{green}+0.00%$
$\color{red}98788/122179$
$\color{green}+0.00%$
blas/ext/base/ndarray/dcusum $\color{green}123/123$
$\color{green}+0.00%$
$\color{green}3/3$
$\color{green}+0.00%$
$\color{green}1/1$
$\color{green}+0.00%$
$\color{green}123/123$
$\color{green}+0.00%$
blas/ext/base/ndarray/dcusumkbn $\color{green}123/123$
$\color{green}+0.00%$
$\color{green}3/3$
$\color{green}+0.00%$
$\color{green}1/1$
$\color{green}+0.00%$
$\color{green}123/123$
$\color{green}+0.00%$
blas/ext/base/ndarray/dcusumkbn2 $\color{green}123/123$
$\color{green}+0.00%$
$\color{green}3/3$
$\color{green}+0.00%$
$\color{green}1/1$
$\color{green}+0.00%$
$\color{green}123/123$
$\color{green}+0.00%$
blas/ext/base/ndarray/dcusumors $\color{green}123/123$
$\color{green}+0.00%$
$\color{green}3/3$
$\color{green}+0.00%$
$\color{green}1/1$
$\color{green}+0.00%$
$\color{green}123/123$
$\color{green}+0.00%$
blas/ext/base/ndarray/dcusumpw $\color{green}123/123$
$\color{green}+0.00%$
$\color{green}3/3$
$\color{green}+0.00%$
$\color{green}1/1$
$\color{green}+0.00%$
$\color{green}123/123$
$\color{green}+0.00%$
datasets/us-states-capitals-names $\color{green}146/146$
$\color{green}+0.00%$
$\color{green}7/7$
$\color{green}+0.00%$
$\color{green}2/2$
$\color{green}+0.00%$
$\color{green}146/146$
$\color{green}+0.00%$
stats/base/ndarray/cumin $\color{green}113/113$
$\color{green}+0.00%$
$\color{green}3/3$
$\color{green}+0.00%$
$\color{green}1/1$
$\color{green}+0.00%$
$\color{green}113/113$
$\color{green}+0.00%$
utils/async/for-each $\color{green}525/525$
$\color{green}+0.00%$
$\color{green}59/59$
$\color{green}+0.00%$
$\color{green}8/8$
$\color{green}+0.00%$
$\color{green}525/525$
$\color{green}+0.00%$
utils/async/inmap-right $\color{green}542/542$
$\color{green}+0.00%$
$\color{green}62/62$
$\color{green}+0.00%$
$\color{green}9/9$
$\color{green}+0.00%$
$\color{green}542/542$
$\color{green}+0.00%$
utils/async/inmap $\color{green}544/544$
$\color{green}+0.00%$
$\color{green}62/62$
$\color{green}+0.00%$
$\color{green}9/9$
$\color{green}+0.00%$
$\color{green}544/544$
$\color{green}+0.00%$

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

@Planeshifter Planeshifter changed the title Propagate fixes merged to develop on 2026-04-19 chore: propagate recent doctest and lint fixes to sibling packages Apr 19, 2026
@Planeshifter Planeshifter marked this pull request as ready for review April 19, 2026 16:53
@Planeshifter Planeshifter requested review from a team and kgryte April 19, 2026 16:53
@stdlib-bot stdlib-bot added the Needs Review A pull request which needs code review. label Apr 19, 2026
@kgryte kgryte removed the Needs Review A pull request which needs code review. label Apr 19, 2026
Comment thread lib/node_modules/@stdlib/utils/async/for-each/examples/index.js
Comment thread lib/node_modules/@stdlib/utils/async/inmap-right/examples/index.js
Comment thread lib/node_modules/@stdlib/utils/async/inmap/examples/index.js
Extends 8adce00 to the matching README example blocks. Athan Reines flagged that the `examples/index.js` fix needed to land in the READMEs too; updates `utils/async/inmap`, `utils/async/inmap-right`, `utils/async/for-each`, and `datasets/us-states-capitals-names` README examples to use `@stdlib/string/format` instead of string concatenation inside `new Error(...)`.
@Planeshifter Planeshifter requested a review from kgryte April 19, 2026 19:14
@stdlib-bot stdlib-bot added the Needs Review A pull request which needs code review. label Apr 19, 2026
@kgryte kgryte removed the Needs Review A pull request which needs code review. label Apr 19, 2026
@kgryte kgryte merged commit 67cdf3c into develop Apr 19, 2026
32 checks passed
@kgryte kgryte deleted the philipp/fix-propagation-2026-04-19 branch April 19, 2026 19:18
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.

4 participants