Skip to content

chore: propagate fixes across blas/ext/base and regexp#12970

Merged
kgryte merged 2 commits into
developfrom
philipp/fix-propagation-2026-06-19
Jun 20, 2026
Merged

chore: propagate fixes across blas/ext/base and regexp#12970
kgryte merged 2 commits into
developfrom
philipp/fix-propagation-2026-06-19

Conversation

@Planeshifter

Copy link
Copy Markdown
Member

Description

Propagating fixes merged to develop between 2026-06-18 17:52 UTC and 2026-06-19 06:03 UTC to sibling packages.

docs: fix capitalization across blas/ext/base``

Propagates a1c00d0, which corrected a sentence-initial lowercase if to If in the REPL help text for blas/ext/base/gindex-of-falsy. The same malformed sentence appeared verbatim in six sibling packages and is fixed here uniformly.

Source: a1c00d0

Targets:

  • blas/ext/base/dindex-of-truthy
  • blas/ext/base/dindex-of
  • blas/ext/base/gindex-of-truthy
  • blas/ext/base/gindex-of
  • blas/ext/base/sindex-of-truthy
  • blas/ext/base/sindex-of

test: remove redundant filename output in regexp/color-hexadecimal``

Removes three redundant t.ok( true, __filename ); assertions from non-first tape(...) blocks in regexp/color-hexadecimal/test/test.js. Per the stdlib/first-unit-test ESLint rule, only the leading test block carries the filename assertion; repeating it in subsequent blocks adds no signal. A second candidate (math/base/special/ellipj/test/test.sncndn.js) was excluded because that file carries an explicit /* eslint-disable stdlib/first-unit-test */ directive.

Source: f4bc442

Targets:

  • regexp/color-hexadecimal

Related Issues

No related issues.

Questions

No.

Other

Validation

  • Pattern search scoped to sibling directories of each source commit (blas/ext/base/*/docs/repl.txt for the capitalization fix; all lib/node_modules/@stdlib/**/test/*.js with multiple t.ok( true, __filename ); occurrences for the redundant-assertion fix).
  • Two independent validation agents read each candidate file in full and confirmed the defect; a style-consistency pass confirmed indentation and surrounding-line integrity.
  • Excluded from this PR:
    • math/base/special/ellipj/test/test.sncndn.js — file carries /* eslint-disable stdlib/first-unit-test */, indicating the author opted out of the convention.
    • math/base/special/xlogy/test/test.native.js — flagged needs-human; the duplicate t.ok( true, __filename ); lines reflect a duplicated tape(...) block rather than the redundant-assertion pattern.
    • _tools/eslint/rules/first-unit-test/** — the lint rule's own fixtures and implementation; intentionally retain matching strings.

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

This PR was authored by Claude Code running an automated fix-propagation routine: candidate fixes from the prior 24h on develop were specified, sibling sites were located by pattern search, then two independent validation agents and a style-consistency agent confirmed each target before patches were applied.


@stdlib-js/reviewers


Generated by Claude Code

claude added 2 commits June 19, 2026 13:11
Propagates fix from a1c00d0 ("docs: fix capitalization") to sibling
packages whose REPL docs had the same sentence-starting lowercase `if`
preceded by a blank line.
Propagates fix from f4bc442 ("test: remove redundant filename output in
test blocks") to a sibling test file whose non-first `tape(...)` blocks
contained the same redundant `t.ok( true, __filename );` assertion that
the `stdlib/first-unit-test` lint rule discourages.
@stdlib-bot

Copy link
Copy Markdown
Contributor

Coverage Report

Package Statements Branches Functions Lines
blas/ext/base/dindex-of-truthy $\\color{green}373/373$
$\\color{green}+100.00\\%$
$\\color{green}19/19$
$\\color{green}+100.00\\%$
$\\color{green}4/4$
$\\color{green}+100.00\\%$
$\\color{green}373/373$
$\\color{green}+100.00\\%$
blas/ext/base/dindex-of $\\color{green}357/357$
$\\color{green}+100.00\\%$
$\\color{green}19/19$
$\\color{green}+100.00\\%$
$\\color{green}4/4$
$\\color{green}+100.00\\%$
$\\color{green}357/357$
$\\color{green}+100.00\\%$
blas/ext/base/gindex-of-truthy $\\color{green}257/257$
$\\color{green}+100.00\\%$
$\\color{green}18/18$
$\\color{green}+100.00\\%$
$\\color{green}3/3$
$\\color{green}+100.00\\%$
$\\color{green}257/257$
$\\color{green}+100.00\\%$
blas/ext/base/gindex-of $\\color{green}248/248$
$\\color{green}+100.00\\%$
$\\color{green}18/18$
$\\color{green}+100.00\\%$
$\\color{green}3/3$
$\\color{green}+100.00\\%$
$\\color{green}248/248$
$\\color{green}+100.00\\%$
blas/ext/base/sindex-of-truthy $\\color{green}373/373$
$\\color{green}+100.00\\%$
$\\color{green}19/19$
$\\color{green}+100.00\\%$
$\\color{green}4/4$
$\\color{green}+100.00\\%$
$\\color{green}373/373$
$\\color{green}+100.00\\%$
blas/ext/base/sindex-of $\\color{green}357/357$
$\\color{green}+100.00\\%$
$\\color{green}19/19$
$\\color{green}+100.00\\%$
$\\color{green}4/4$
$\\color{green}+100.00\\%$
$\\color{green}357/357$
$\\color{green}+100.00\\%$
regexp/color-hexadecimal $\\color{green}173/173$
$\\color{green}+100.00\\%$
$\\color{green}9/9$
$\\color{green}+100.00\\%$
$\\color{green}1/1$
$\\color{green}+100.00\\%$
$\\color{green}173/173$
$\\color{green}+100.00\\%$

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

@kgryte kgryte marked this pull request as ready for review June 20, 2026 21:37
@kgryte kgryte requested a review from a team June 20, 2026 21:38
@stdlib-bot stdlib-bot added the Needs Review A pull request which needs code review. label Jun 20, 2026
@kgryte kgryte changed the title docs: propagate develop fixes across blas/ext/base and regexp chore: propagate fixes across blas/ext/base and regexp Jun 20, 2026
@kgryte kgryte merged commit aa598c9 into develop Jun 20, 2026
35 checks passed
@kgryte kgryte deleted the philipp/fix-propagation-2026-06-19 branch June 20, 2026 21:38
@kgryte kgryte removed the Needs Review A pull request which needs code review. label Jun 20, 2026
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