Skip to content

Commit

Permalink
CI: amend #306
Browse files Browse the repository at this point in the history
- I only installed npcap, but forgot to run the actual tests 🤦
  • Loading branch information
cyqsimon committed Oct 16, 2023
1 parent df17172 commit 5d2ee96
Showing 1 changed file with 3 additions and 3 deletions.
6 changes: 3 additions & 3 deletions .github/workflows/ci.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -88,9 +88,9 @@ jobs:
- name: Run tests using cargo-insta
id: run_tests
# npcap is needed to run tests on Windows, so this is an unfortunate
# sacrifice we have to make in the name of security
if: (matrix.os != 'windows-latest') || (github.event_name != 'pull_request')
# npcap is needed to run tests on Windows, so unfortunately we cannot run tests
# on PRs from other repositories
if: matrix.os != 'windows-latest' || github.event_name != 'pull_request' || github.event.pull_request.head.repo.full_name == github.repository
env:
# make insta generate new snapshots in CI
INSTA_UPDATE: new
Expand Down

0 comments on commit 5d2ee96

Please sign in to comment.