chore: propagate fixes to sibling packages#11892
Merged
Conversation
…ription Propagates fix from e67e59e ("docs: add missing period and align keywords in `constants/float16`") to the float32 sibling, whose description likewise omitted the trailing period. https://claude.ai/code/session_01QJsHa2HjbAGpNE1xuYAJ6D
Propagates fix from 87038a4 ("chore: fix EditorConfig lint errors") to twenty additional `math/base/special/*` and `math/base/tools/*` test fixtures whose final byte is not a newline, restoring conformance with the `insert_final_newline` EditorConfig rule. https://claude.ai/code/session_01QJsHa2HjbAGpNE1xuYAJ6D
Contributor
Coverage Report
The above coverage report was generated for the changes in this PR. |
develop on 2026-05-02 to sibling packages
kgryte
approved these changes
May 3, 2026
Member
|
@Planeshifter This PR fixes the missing newline in the fixture files; however, this is actually a symptom of the test fixture runner which needs to be updated to append a trailing newline to the output. Maybe something to address in a follow-up PR. I believe all our snippets should now append a trailing newline (e.g., Julia). |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Description
Propagating fixes merged to
developbetween 2026-05-01 17:21 -0400 and 2026-05-02 09:37 +0000 to sibling packages exhibiting the same defect.e67e59e— trailing period inpackage.jsondescriptionPropagates the trailing-period fix from
e67e59etoconstants/float32/ln-half, where thedescriptionfield had the same missing punctuation. Keeps description style consistent with the rest of the constants packages.lib/node_modules/@stdlib/constants/float32/ln-half/package.json87038a4— final newline in JSON test fixturesFollows
87038a4, which fixed a missing final newline in onesectest fixture; the same EditorConfiginsert_final_newlinedefect was present in 20 sibling fixtures acrossmath/base/special/acscdf,math/base/special/asecdf,math/base/special/cosm1,math/base/special/sec, andmath/base/tools/normhermitepolyf. All affected JSON fixture files under those paths have been brought into compliance.lib/node_modules/@stdlib/math/base/special/acscdf/test/fixtures/julia/{negative,positive}.jsonlib/node_modules/@stdlib/math/base/special/asecdf/test/fixtures/julia/{negative,positive}.jsonlib/node_modules/@stdlib/math/base/special/cosm1/test/fixtures/python/data.jsonlib/node_modules/@stdlib/math/base/special/sec/test/fixtures/julia/*.json(9 sibling fixtures)lib/node_modules/@stdlib/math/base/tools/normhermitepolyf/test/fixtures/python/*.json(6 fixtures)Related Issues
No.
Questions
No.
Other
Validation
lib/node_modules/@stdlib/constants/**/package.json(250 files scanned) for the description-period pattern;lib/node_modules/@stdlib/**/*.json(binary last-byte check) for the final-newline pattern.constants/float32/*; trailing newline on already-fixed fixtures inmath/base/special/sec/).Deliberately excluded
fd45d7b(JSDoc throws ordering acrossstats/array) — exhaustive search acrossstats/base/,stats/strided/,blas/,random/array/,random/strided/(4,999main.jsfiles scanned) returned zero matches; the source commit was already exhaustive.394c99f(missing README stub sections) — 129 candidate files surfaced, but namespace-level READMEs (e.g.array/base/README.md,array/base/assert/README.md) were dropped because no namespace-level README instring/base,math/base,stats/base,blas/base,ndarray/base, orarray/basecarries a<section class="references">stub; this is a leaf-package convention. The remaining 127 leaf candidates were left to a follow-up sweep so this PR remains tightly scoped to direct propagation.c7a6b5c(JS lint sub-fixes) — Float64Array array-literal spacing has both styles in active use across the repo; the nested-function hoisting fix requires AST analysis; the JSDoc example value fix requires semantic analysis.e67e59ekeyword alignment — namespace canonical keyword sets vary; not mechanically derivable.datasets/moby-dick/*.json,datasets/spam-assassin/*.json) — 6,187 bulk dataset files lack final newlines but are a separate remediation scope from generated test fixtures.No sites were flagged
needs-humanand forced through. Pattern C was dropped entirely after both validation agent 2 and the style-consistency agent independently flagged the namespace-vs-leaf distinction.Checklist
AI Assistance
If you answered "yes" above, how did you use AI assistance?
Disclosure
This PR was authored by Claude Code running an automated fix-propagation routine: scanning the last 24 hours of commits to
developfor fixes following a generalizable pattern, searching for sibling packages with the same defect, independently validating each candidate site with parallel agents, and applying the equivalent fix. A maintainer should audit each propagation site before promoting from draft.Generated by Claude Code