Skip to content

Commit 6f3a523

Browse files
committed
chore: remove retry logic from workflow
The retry logic was a workaround for the hanging command issue. With the fixed implementation using shutil.which(), we don't need it.
1 parent e0afc30 commit 6f3a523

File tree

1 file changed

+3
-15
lines changed

1 file changed

+3
-15
lines changed

.github/workflows/test.yml

Lines changed: 3 additions & 15 deletions
Original file line numberDiff line numberDiff line change
@@ -72,9 +72,9 @@ jobs:
7272
steps:
7373
- uses: actions/checkout@v6
7474

75-
- uses: actions/setup-node@v4
75+
- uses: actions/setup-node@v6
7676
with:
77-
node-version: "20"
77+
node-version: "24"
7878

7979
- uses: astral-sh/setup-uv@v7
8080
with:
@@ -86,20 +86,8 @@ jobs:
8686
- name: Install package
8787
run: uv sync
8888

89-
- name: Set custom npm cache directory on Windows
90-
if: runner.os == 'Windows'
91-
run: |
92-
echo "NPM_CONFIG_CACHE=${{ runner.temp }}\npm-cache" >> $GITHUB_ENV
93-
npm config set cache "${{ runner.temp }}\npm-cache" --global
94-
shell: bash
95-
9689
- name: Test CLI can be invoked
97-
uses: nick-fields/retry@v3
98-
with:
99-
timeout_minutes: 5
100-
max_attempts: 2
101-
retry_on: error
102-
command: uv run promptfoo --version
90+
run: uv run promptfoo --version
10391

10492
- name: Test Node.js detection
10593
run: uv run python -c "from promptfoo.cli import check_node_installed, check_npx_installed; assert check_node_installed(); assert check_npx_installed()"

0 commit comments

Comments
 (0)