Skip to content

Commit 7a12239

Browse files
committed
revert: build: revert temporary CI workflow changes
Revert the affected-tests workflow modifications (namespace CLI test exclusion and fixture-only fan-out skip) that were added to work around CI timeouts while validating the R 4.2 upgrade. The substantive changes have now passed CI successfully. Assisted-by: Claude Opus 4.6 <noreply@anthropic.com> --- type: pre_commit_static_analysis_report description: Results of running static analysis checks when committing changes. report: - task: lint_filenames status: passed - task: lint_editorconfig status: passed - task: lint_markdown status: na - task: lint_package_json status: na - task: lint_repl_help status: na - task: lint_javascript_src status: na - task: lint_javascript_cli status: na - task: lint_javascript_examples status: na - task: lint_javascript_tests status: na - task: lint_javascript_benchmarks status: na - task: lint_python status: na - task: lint_r status: na - task: lint_c_src status: na - task: lint_c_examples status: na - task: lint_c_benchmarks status: na - task: lint_c_tests_fixtures status: na - task: lint_shell status: missing_dependencies - task: lint_typescript_declarations status: passed - task: lint_typescript_tests status: na - task: lint_license_headers status: passed ---
1 parent 45bb989 commit 7a12239

1 file changed

Lines changed: 16 additions & 23 deletions

File tree

  • .github/workflows/scripts/run_affected_tests

.github/workflows/scripts/run_affected_tests/run

Lines changed: 16 additions & 23 deletions
Original file line numberDiff line numberDiff line change
@@ -122,30 +122,23 @@ main() {
122122
# Extract package names from changed package directories (e.g., @stdlib/math/base/special/sin) by removing the leading 'lib/node_modules/':
123123
packages=$(echo "${directories}" | sed -E 's/^lib\/node_modules\///')
124124

125-
# Determine whether any non-fixture files changed. If only test fixtures were modified, skip dependency fan-out since downstream packages cannot be affected:
126-
non_fixture_files=$(echo "${changed}" | tr ' ' '\n' | grep -v '/test/fixtures/') || true
127-
128-
if [ -n "${non_fixture_files}" ]; then
129-
# Find all package directories which `require()` one of the changed packages:
130-
required_by=""
131-
for package in ${packages}; do
132-
echo "Finding packages which depend on '${package}'..."
133-
escaped_package=$(echo "$package" | sed 's/[\/&]/\\&/g')
134-
dependents=$(find lib/node_modules/@stdlib -type f -name '*.js' -exec grep -ol -E "require\( [']${escaped_package}['] \)" {} +) || true
135-
echo "Found: ${dependents}"
136-
if [ -n "${dependents}" ]; then
137-
dependents=$(dirname $dependents | sed -E 's/\/(benchmark|bin|data|docs|etc|examples|include|lib|scripts|src|test)(\/.*)?$//' | sort -u)
138-
echo "List of dependents: ${dependents}"
139-
required_by="${required_by} ${dependents}"
140-
fi
141-
done
142-
143-
# Concatenate the list of changed package directories and package directories which `require()` one of the changed packages:
144-
if [ -n "${required_by}" ]; then
145-
directories="${directories} ${required_by}"
125+
# Find all package directories which `require()` one of the changed packages:
126+
required_by=""
127+
for package in ${packages}; do
128+
echo "Finding packages which depend on '${package}'..."
129+
escaped_package=$(echo "$package" | sed 's/[\/&]/\\&/g')
130+
dependents=$(find lib/node_modules/@stdlib -type f -name '*.js' -exec grep -ol -E "require\( [']${escaped_package}['] \)" {} +) || true
131+
echo "Found: ${dependents}"
132+
if [ -n "${dependents}" ]; then
133+
dependents=$(dirname $dependents | sed -E 's/\/(benchmark|bin|data|docs|etc|examples|include|lib|scripts|src|test)(\/.*)?$//' | sort -u)
134+
echo "List of dependents: ${dependents}"
135+
required_by="${required_by} ${dependents}"
146136
fi
147-
else
148-
echo 'Only test fixture files changed. Skipping dependency fan-out.' >&2
137+
done
138+
139+
# Concatenate the list of changed package directories and package directories which `require()` one of the changed packages:
140+
if [ -n "${required_by}" ]; then
141+
directories="${directories} ${required_by}"
149142
fi
150143

151144
# Build native add-ons for packages (if applicable):

0 commit comments

Comments
 (0)