Repository navigation
docs: update type - #16330
docs: update type#16330
Conversation
Propagates fix from 7bf6853 ("chore: clean-up") to sibling packages documenting `N` as `PositiveInteger` despite accepting `N = 0` as a no-op. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_017wWD78YjA2bMx3817c7htZ
Coverage Report
The above coverage report was generated for the changes in this PR. |
Ports the fix from #16328 to resolve the CI dependency installation failure (`binding.gyp not found` when building `@kaciras/deasync`). No-op once the base branch carries the same change. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_017wWD78YjA2bMx3817c7htZ
|
I ported the pin from #16328 ( Generated by Claude Code |
|
The gh54 ESLint migration renamed -var defaults = require( './../../eslint.flat.config.js' );
+var defaults = require( './../../eslint.config.js' );Re-running would fail the same way, so I did not re-run. Generated by Claude Code |
|
Both remaining red checks clear on the next push whose range is limited to this PR's files: this one and Generated by Claude Code |
N parameter type in lapack/base/* JSDoc
Description
This pull request:
developbetween 2026-10-26 13:34 UTC and 2026-10-27 13:34 UTC to sibling packages.NJSDoc type correction from 7bf6853 (chore: clean-up), which retypedNasNonNegativeIntegerinlapack/base/*laswp. The samePositiveIntegerannotation remained in fourlapack/basepackages whosendarraykernels return early whenN <= 0, as reference LAPACK does, and whose tests checkN = 0. Target packages:lapack/base/clacgvlapack/base/zlacgvlapack/base/crotlapack/base/zrotRelated Issues
None.
Reference Packages
This pull request followed these packages:
lapack/base/dlaswp{NonNegativeInteger} NJSDoc type from 7bf6853.Questions
No.
Other
Validation: I searched for
@param {PositiveInteger} Nacrosslapack/base/*/lib. Two independent validation passes read each target file and itsndarraycallee, and both confirmed all 8 sites. A runtime check withN = 0andN = -1returned the inputs unchanged. A style and adaptation pass confirmed the change applies verbatim, with no cascade intodocs/types, README,repl.txt, or tests. After this change, nolapack/base/*/libfile documentsNasPositiveInteger.Excluded:
format()wrapper on thrown error messages, also from 7bf6853, still appears inblas/base/{dsymv,sspmv}/lib/*.js(22 sites). These were dropped because open PRs refactor: update the implementation ofblas/base/dsymv#11087 and refactor: update implementation forblas/base/sspmv#2839 already addformat()in the same files.{PositiveInteger} N - number of indexed elementsannotations remain underblas/*andblas/ext/*. Retyping them would be a much broader sweep than the source commit and should be decided separately.@moduleexample variable-name mismatch from 7bf6853 has no remaining sites.Checklist
AI Assistance
If you answered "yes" above, how did you use AI assistance?
Disclosure
This PR was written primarily by Claude Code as part of an automated fix-propagation routine.
@stdlib-js/reviewers
Generated by Claude Code