Skip to content

Commit

Permalink
Merge pull request #13 from storyprotocol/fix/workflow_syntax
Browse files Browse the repository at this point in the history
[fix] syntax issue
  • Loading branch information
AndyBoWu authored Mar 29, 2024
2 parents df901eb + 4303395 commit 1715226
Showing 1 changed file with 8 additions and 9 deletions.
17 changes: 8 additions & 9 deletions .github/workflows/ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -44,19 +44,18 @@ jobs:
run: pnpm eslint .
# Ensure you have a lint script in your package.json

- name: Set env
env:
API_BASE_URL: ${{ vars.API_BASE_URL }}
API_PREFIX: ${{ vars.API_PREFIX }}
API_KEY: ${{ secrets.API_KEY }}

- name: Run Tests
run: pnpm exec playwright test
# Ensure you have a test:integration script in your package.json for your API integration tests
continue-on-error: true
run: |
echo "Load environment variables"
export API_BASE_URL=${{ secrets.API_BASE_URL }}
export API_PREFIX=${{ secrets.API_PREFIX }}
export API_KEY=${{ secrets.API_KEY }}
pnpm exec playwright test
- name: Upload Test Results
uses: actions/upload-artifact@v2
if: failure() && github.event_name == 'push' && github.ref == 'refs/heads/main'
# if: failure() && github.event_name == 'push' && github.ref == 'refs/heads/main'
with:
name: test-results
path: ./playwright-report/index.html

0 comments on commit 1715226

Please sign in to comment.