Skip to content

Commit 9305069

Browse files
stdlib-botstdlib-bot
committed
Merge remote-tracking branch 'upstream/develop' into feat/add-uint64array-findindex
2 parents 25f5609 + 9f36f33 commit 9305069

10,106 files changed

Lines changed: 779613 additions & 52181 deletions

File tree

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

‎.github/workflows/process_metadata.yml‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -117,7 +117,7 @@ jobs:
117117
env:
118118
METADATA: ${{ steps.extract-metadata.outputs.metadata }}
119119
run: |
120-
inputs=$(echo "$METADATA" | jq -c '.[] | select(.type | contains("workflow_dispatch"))')
120+
inputs=$(echo "$METADATA" | jq -c '.[] | select((.type // "") | contains("workflow_dispatch"))')
121121
if [ -n "$inputs" ]; then
122122
name=$(echo $inputs | jq -r '.name')
123123
inputs=$(echo $inputs | jq -c 'del(.type, .name, .author, .id, .url) | map_values(tostring)')

‎.github/workflows/random_benchmarks.yml‎

Lines changed: 7 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -86,7 +86,13 @@ jobs:
8686
- name: 'Run JavaScript and native add-on benchmarks'
8787
run: |
8888
make benchmark-random-javascript RANDOM_SELECTION_SIZE=50 BUILD_ADDONS=1
89-
timeout-minutes: 60
89+
timeout-minutes: 90
90+
91+
# Install Cephes dependencies (required by C benchmarks which compare against the Cephes math library; accounting for possible network failures when downloading):
92+
- name: 'Install Cephes dependencies'
93+
run: |
94+
make install-deps-cephes || make install-deps-cephes || make install-deps-cephes
95+
timeout-minutes: 15
9096

9197
# Run C benchmarks:
9298
- name: 'Run C benchmarks'

‎.github/workflows/run_tests_coverage_pr.yml‎

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -120,9 +120,9 @@ jobs:
120120
id: changed-directories
121121
continue-on-error: true
122122
run: |
123-
# Get changed files using git diff against base branch:
124-
git fetch origin ${{ github.base_ref }} --depth=1
125-
files=$(git diff --diff-filter=AM --name-only origin/${{ github.base_ref }}...HEAD)
123+
# Get the list of changed files in pull request:
124+
ancestor_commit=$(git merge-base ${{ github.event.pull_request.base.sha }} ${{ github.event.pull_request.head.sha }})
125+
files=$(git diff --diff-filter=AM --name-only $ancestor_commit ${{ github.event.pull_request.head.sha }})
126126
directories=$(for file in $files; do dirname $file; done | sort -u | tr '\n' ' ' | sed 's/ $//')
127127
echo "directories=${directories}" >> $GITHUB_OUTPUT
128128

‎.github/workflows/scripts/check_duplicate_prs/run‎

Lines changed: 4 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -255,14 +255,12 @@ main() {
255255
echo "Processing PRs for issue references..."
256256
debug_log "Starting to process $total_prs PRs for issue references"
257257

258-
# Process PRs one by one...
258+
# Process PRs one by one, streaming the list through a single `jq` invocation in order to avoid re-parsing the entire list once per PR...
259259
processed_count=0
260260

261-
for ((i=0; i<total_prs; i++)); do
262-
pr=$(echo "$open_prs" | jq -c ".[$i]")
263-
261+
while IFS= read -r pr; do
264262
if [ -z "$pr" ] || [ "$pr" = "null" ]; then
265-
debug_log "Warning: Empty PR data at index $i"
263+
debug_log "Warning: Empty PR data"
266264
continue
267265
fi
268266

@@ -300,7 +298,7 @@ main() {
300298
issue_prs_values[index]="${issue_prs_values[index]} $pr_number"
301299
fi
302300
done
303-
done
301+
done < <(echo "$open_prs" | jq -c '.[]')
304302

305303
debug_log "Finished processing all PRs for issue references"
306304
debug_log "Found ${#issue_prs_keys[@]} unique issues referenced in PRs"

‎.github/workflows/scripts/run_affected_benchmarks/run‎

Lines changed: 11 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -129,20 +129,20 @@ main() {
129129
fi
130130
done
131131

132-
# Find all benchmark files in package directories:
132+
# Find all benchmark files in package directories (note: benchmark discovery is intentionally scoped to each package's own `benchmark` directory so that changes to a namespace package do not trigger benchmarks for that namespace's sub-packages):
133133
js_bench_files=""
134134
for dir in ${directories}; do
135-
if [ -d "${dir}" ]; then
136-
files=$(find "${dir}" -maxdepth 3 -wholename '**/benchmark/benchmark*.js' | grep -v '/fixtures/' | sort -u | tr '\n' ' ') || true
135+
if [ -d "${dir}/benchmark" ]; then
136+
files=$(find "${dir}/benchmark" -name 'benchmark*.js' | grep -v '/fixtures/' | sort -u | tr '\n' ' ') || true
137137
js_bench_files="${js_bench_files} ${files}"
138138
fi
139139
done
140140
js_bench_files=$(echo "${js_bench_files}" | xargs)
141141

142142
# Run JS benchmarks:
143143
if [ -n "${js_bench_files}" ]; then
144-
# Invoke make in batches to avoid "Argument list too long" errors:
145-
printf '%s\n' ${js_bench_files} | xargs sh -c 'make benchmark-javascript-files FILES="$*"' _
144+
# Invoke make in batches, capped well under the 128KiB Linux single-argument limit, in order to avoid "Argument list too long" errors:
145+
printf '%s\n' ${js_bench_files} | xargs -s 40000 sh -c 'make benchmark-javascript-files FILES="$*"' _
146146
else
147147
echo 'No JavaScript benchmarks to run.' >&2
148148
fi
@@ -151,8 +151,8 @@ main() {
151151
echo "Finding C benchmark files in ${directories}..."
152152
c_bench_files=""
153153
for dir in ${directories}; do
154-
if [ -d "${dir}" ]; then
155-
files=$(find "${dir}" -maxdepth 5 \( -wholename '**/benchmark/c/benchmark*.c' -or -wholename '**/benchmark/c/**/benchmark*.c' \) -exec realpath {} \; | grep -v '/fixtures/' | sort -u | tr '\n' ' ') || true
154+
if [ -d "${dir}/benchmark/c" ]; then
155+
files=$(find "${dir}/benchmark/c" -name 'benchmark*.c' -exec realpath {} \; | grep -v '/fixtures/' | sort -u | tr '\n' ' ') || true
156156
c_bench_files="${c_bench_files} ${files}"
157157
fi
158158
done
@@ -161,8 +161,8 @@ main() {
161161
# If benchmarks requiring Cephes are found, install the Cephes library:
162162
c_cephes_makefiles=""
163163
for dir in ${directories}; do
164-
if [ -d "${dir}" ]; then
165-
files=$(find "${dir}" -maxdepth 5 -wholename '**/benchmark/c/**/Makefile' -exec grep -l 'CEPHES' {} + || true)
164+
if [ -d "${dir}/benchmark/c" ]; then
165+
files=$(find "${dir}/benchmark/c" -name 'Makefile' -exec grep -l 'CEPHES' {} + || true)
166166
c_cephes_makefiles="${c_cephes_makefiles} ${files}"
167167
fi
168168
done
@@ -173,8 +173,8 @@ main() {
173173
fi
174174

175175
if [ -n "${c_bench_files}" ]; then
176-
# Invoke make in batches to avoid "Argument list too long" errors:
177-
printf '%s\n' ${c_bench_files} | xargs sh -c 'make benchmark-c-files FILES="$*"' _
176+
# Invoke make in batches, capped well under the 128KiB Linux single-argument limit, in order to avoid "Argument list too long" errors:
177+
printf '%s\n' ${c_bench_files} | xargs -s 40000 sh -c 'make benchmark-c-files FILES="$*"' _
178178
else
179179
echo 'No C benchmarks to run.' >&2
180180
fi

‎.github/workflows/scripts/run_affected_tests/run‎

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -168,8 +168,8 @@ main() {
168168
files=$(echo "${files}" | grep -v '/fixtures/') || true
169169

170170
if [[ -n "${files}" ]]; then
171-
# Invoke make in batches to avoid "Argument list too long" errors:
172-
printf '%s\n' ${files} | xargs sh -c 'make test-javascript-files-min FILES="$*"' _
171+
# Invoke make in batches, capped well under the 128KiB Linux single-argument limit, in order to avoid "Argument list too long" errors:
172+
printf '%s\n' ${files} | xargs -s 40000 sh -c 'make test-javascript-files-min FILES="$*"' _
173173
fi
174174

175175
cleanup

‎.github/workflows/scripts/run_tests_coverage/run‎

Lines changed: 14 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -138,12 +138,17 @@ main() {
138138
exit 0
139139
fi
140140

141-
mkdir -p artifacts
142141
coverage=''
143142
for package in ${directories}; do
144143
# For each package, extract coverage values from the respective coverage report:
145144
pkg=$(echo "$package" | sed -E 's/^.*stdlib\///')
146145

146+
# Skip packages which do not have a test directory, as `make test-javascript-cov` fails when a `TESTS_FILTER` matches zero files:
147+
if [ ! -d "lib/node_modules/@stdlib/${pkg}/test" ]; then
148+
echo "WARNING: No test directory found for package '${pkg}'. Skipping..." >&2
149+
continue
150+
fi
151+
147152
if [ -f "lib/node_modules/@stdlib/${pkg}/binding.gyp" ]; then
148153
NODE_ADDONS_PATTERN="${pkg}" make install-node-addons
149154
fi
@@ -195,10 +200,14 @@ main() {
195200
rm -rf reports/coverage/lcov-report/*
196201
done
197202

198-
# Format coverage as Markdown table row:
199-
table_body=$(echo "$coverage" | sed -e 's/,/|/g; s/"/ /g; s/\[/|/g; s/\]/|/g')
200-
table_header="| Package | Statements | Branches | Functions | Lines |\n| --------- | ------------ | ---------- | ----------- | ----- |"
201-
table="${table_header}${table_body}"
203+
# Format coverage as a Markdown table row, unless every changed package was skipped (e.g., no package had a test directory):
204+
if [ -z "$coverage" ]; then
205+
table=''
206+
else
207+
table_body=$(echo "$coverage" | sed -e 's/,/|/g; s/"/ /g; s/\[/|/g; s/\]/|/g')
208+
table_header="| Package | Statements | Branches | Functions | Lines |\n| --------- | ------------ | ---------- | ----------- | ----- |"
209+
table="${table_header}${table_body}"
210+
fi
202211

203212
echo "table=$table" >> "$GITHUB_OUTPUT"
204213

‎.mailmap‎

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -272,7 +272,8 @@ Pranav Goswami <pranavchiku11@gmail.com> Pranav
272272

273273
Pranjal Jha <97080887+PraneGIT@users.noreply.github.com> PraneGIT
274274

275-
Prashant Kumar Yadav <144602492+0PrashantYadav0@users.noreply.github.com> JoyBoy
275+
Prashant Kumar Yadav <devprashantkyadav@gmail.com> <144602492+0PrashantYadav0@users.noreply.github.com>
276+
Prashant Kumar Yadav <devprashantkyadav@gmail.com> JoyBoy
276277

277278
Pratik Singh <97464067+Pratik772846@users.noreply.github.com> Pratik772846
278279

‎.rtlintrc‎

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -6,6 +6,7 @@
66
"@stdlib/_tools/repl-txt/rules/interface-spacing": [ "error", 2 ],
77
"@stdlib/_tools/repl-txt/rules/line-length": [ "error", 80 ],
88
"@stdlib/_tools/repl-txt/rules/match-signature": "error",
9+
"@stdlib/_tools/repl-txt/rules/no-redeclare": "error",
910
"@stdlib/_tools/repl-txt/rules/resolvable-aliases": "warn",
1011
"@stdlib/_tools/repl-txt/rules/section-order": "error",
1112
"@stdlib/_tools/repl-txt/rules/section-heading-style": "error",

‎CONTRIBUTORS‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -159,7 +159,7 @@ Prajwal Kulkarni <prajwalkulkarni76@gmail.com>
159159
Pranav Goswami <pranavchiku11@gmail.com>
160160
pranav-1720 <123018993+pranav-1720@users.noreply.github.com>
161161
Pranjal Jha <97080887+PraneGIT@users.noreply.github.com>
162-
Prashant Kumar Yadav <144602492+0PrashantYadav0@users.noreply.github.com>
162+
Prashant Kumar Yadav <devprashantkyadav@gmail.com>
163163
PrathamBhamare <164445568+PrathamBhamare@users.noreply.github.com>
164164
Pratik Singh <97464067+Pratik772846@users.noreply.github.com>
165165
Pratyush Kumar Chouhan <pratyushkumar0308@gmail.com>

0 commit comments

Comments
 (0)