Skip to content

ci: add retry logic for npm install and skip scripts in license scan #295

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
carlos-alm merged 3 commits into from
Mar 3, 2026
Merged

ci: add retry logic for npm install and skip scripts in license scan #295

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
b59a18f
ci: add retry logic for npm install and skip scripts in license scan
carlos-alm Mar 3, 2026
d4a51a5
ci: add shell: bash to retry steps for Windows compatibility
carlos-alm Mar 3, 2026
38bed48
ci: add retry logic to license scan npm ci step
carlos-alm Mar 3, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
26 changes: 24 additions & 2 deletions .github/workflows/ci.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -22,7 +22,18 @@ jobs:
node-version: 22

- name: Install dependencies
run: npm install
shell: bash
run: |
for attempt in 1 2 3; do
npm install && break
if [ "$attempt" -lt 3 ]; then
echo "::warning::npm install attempt $attempt failed, retrying in 15s..."
sleep 15
else
echo "::error::npm install failed after 3 attempts"
exit 1
fi
done

- name: Run Biome
run: npx @biomejs/biome check src/ tests/
Expand All @@ -46,7 +57,18 @@ jobs:
node-version: ${{ matrix.node-version }}

- name: Install dependencies
run: npm install
shell: bash
run: |
for attempt in 1 2 3; do
npm install && break
if [ "$attempt" -lt 3 ]; then
echo "::warning::npm install attempt $attempt failed, retrying in 15s..."
sleep 15
else
echo "::error::npm install failed after 3 attempts"
exit 1
fi
done

- name: Run tests
run: npm test
Expand Down
14 changes: 13 additions & 1 deletion .github/workflows/codegraph-impact.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,19 @@ jobs:
- uses: actions/setup-node@v6
with:
node-version: '22'
- run: npm install
- name: Install dependencies
shell: bash
run: |
for attempt in 1 2 3; do
npm install && break
if [ "$attempt" -lt 3 ]; then
echo "::warning::npm install attempt $attempt failed, retrying in 15s..."
sleep 15
else
echo "::error::npm install failed after 3 attempts"
exit 1
fi
done
- uses: actions/cache@v5
with:
path: .codegraph/
Expand Down
13 changes: 12 additions & 1 deletion .github/workflows/shield-license-compliance.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -29,7 +29,18 @@ jobs:
cache: "npm"

- name: Install dependencies
run: npm ci --prefer-offline --no-audit --no-fund
shell: bash
run: |
for attempt in 1 2 3; do
npm ci --prefer-offline --no-audit --no-fund --ignore-scripts && break
if [ "$attempt" -lt 3 ]; then
echo "::warning::npm ci attempt $attempt failed, retrying in 15s..."
sleep 15
else
echo "::error::npm ci failed after 3 attempts"
exit 1
fi
done

- name: Create reports directory
run: mkdir -p license-reports
Expand Down