Skip to content

Commit

Permalink
fix: 🏗 remove windows from runners
Browse files Browse the repository at this point in the history
Only once Snyk LS downloads the CLI itself, we can enable integration tests for windows.
  • Loading branch information
bastiandoetsch committed Apr 20, 2022
1 parent d0a8f58 commit d8f64ed
Showing 1 changed file with 12 additions and 17 deletions.
29 changes: 12 additions & 17 deletions .github/workflows/build.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -53,8 +53,9 @@ jobs:
runs-on: ${{ matrix.os }}
strategy:
matrix:
os: [ ubuntu-latest, macos-latest, windows-latest ]

# we cannot download the CLI on windows
# once we have automatic downloads of the CLI in the LS, we should add Windows runner
os: [ ubuntu-latest, macos-latest ]
steps:
- uses: actions/checkout@v2

Expand All @@ -67,25 +68,19 @@ jobs:
with:
go-version: ${{ env.GO_VERSION }}

- name: Install sha256sum and snyk-macos
if: runner.os == 'macOS'
run: |
curl -sSL https://static.snyk.io/cli/latest/snyk-macos > /usr/local/bin/snyk-macos && chmod +x /usr/local/bin/snyk-macos
- name: Set up Snyk CLI
if: runner.os == 'Linux'
uses: snyk/actions/setup@master

- name: Run integration tests
env:
DEEPROXY_API_URL: ${{secrets.DEEPROXY_API_URL}}
SNYK_TOKEN: ${{secrets.SNYK_TOKEN }}
INTEG_TESTS: 'true'
run: |
make clean test
cleanup-check:
name: cleanup-check
runs-on: ubuntu-latest
needs: integration-tests
steps:
- name: Check if repo left dirty
run: |
GIT_STATUS=$(git status -s)
echo $GIT_STATUS
if [[ "" == $GIT_STATUS ]]
then
exit 0
else
exit 1
fi

0 comments on commit d8f64ed

Please sign in to comment.