Repository navigation
test: remove excess whitespace in descriptions - #16145
Conversation
Propagates fix from ebec084 ("fix: clamp only the upper bound and clean-up") to sibling test files containing the same double space in `tape` descriptions. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01QpKhVP7qEZ3HHEdTbWc7yC
Coverage Report
The above coverage report was generated for the changes in this PR. |
|
The cause is in Re-running would time out again in the same way, so I'm not re-running it, and I'm not widening this PR to change the workflow. One possible fix in the workflow, for a separate PR: only look up dependents for packages that have changed files outside # Only consider packages with changes outside test/benchmark/docs/examples when computing dependents:
src_packages=$(echo "${changed}" | tr ' ' '\n' | grep -vE '/(test|benchmark|docs|examples)/' | sed -E 's/\/(benchmark|bin|data|docs|etc|examples|include|lib|scripts|src|test)(\/.*)?$//' | sort -u | sed -E 's/^lib\/node_modules\///')
for package in ${src_packages}; do
...
doneGenerated by Claude Code |
|
As the CI failure is not due to changes introduced in this PR, I'll go ahead and merge. |
array and string test descriptions
Description
This pull request:
developbetween 2026-10-13T13:00Z and 2026-10-14T13:00Z to sibling packages.tapedescriptions that have a stray double space between words:array/fixed-endian-factory(test/test.includes.js)string/format(test/test.js)string/base/format-interpolate(test/test.js)Related Issues
No.
Questions
No.
Other
Validation
test/*.jsfiles underlib/node_modules/@stdlibfortapedescriptions with consecutive internal spaces. There were three hits, and all three are fixed here.Excluded
clipIndex→clipUpperIndexinndarray/*last-index*): every siblingndarray/*last-index*wrapper already uses the corrected logic.`k`-tuples→Tuplesinndarray/dcartesian-powernotes):ndarray/scartesian-poweralready uses the corrected wording. The strideddcartesian-powerandgcartesian-powerpackages take an explicitkparameter, so the wording there is correct.datasets/moby-dick/datapackage.json): this is a dataset-specific fix with no propagatable source pattern.random/base/minstd-shuffle/test/test.factory.js:1010(r`normalized`): this is a stray-character typo, not the same shape as the source fix, so it is left for a separate change.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.
🤖 Generated with Claude Code
https://claude.ai/code/session_01QpKhVP7qEZ3HHEdTbWc7yC
Generated by Claude Code