From 5ccd4c4c15868f72f36f927248956dfb89ffca59 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Federico=20Kunze=20K=C3=BCllmer?= <31522760+fedekunze@users.noreply.github.com> Date: Tue, 12 Apr 2022 15:51:20 +0200 Subject: [PATCH] ci: cleanup (#1048) --- .github/workflows/build.yml | 8 ++-- .github/workflows/codeql-analysis.yml | 2 +- .github/workflows/deploy-contract.yml | 10 ++-- .github/workflows/linkchecker.yml | 12 ----- .github/workflows/lint.yml | 16 ++++++- .github/workflows/semgrep.yml | 2 +- .github/workflows/test.yml | 67 ++------------------------- 7 files changed, 30 insertions(+), 87 deletions(-) delete mode 100644 .github/workflows/linkchecker.yml diff --git a/.github/workflows/build.yml b/.github/workflows/build.yml index 499b0bb0..5628876a 100644 --- a/.github/workflows/build.yml +++ b/.github/workflows/build.yml @@ -24,10 +24,10 @@ jobs: - uses: technote-space/get-diff-action@v6.0.1 id: git_diff with: - SUFFIX_FILTER: | - .go - .mod - .sum + PATTERNS: | + **/**.go + go.mod + go.sum - run: | make build if: env.GIT_DIFF diff --git a/.github/workflows/codeql-analysis.yml b/.github/workflows/codeql-analysis.yml index 6e67a0c6..a20675d5 100644 --- a/.github/workflows/codeql-analysis.yml +++ b/.github/workflows/codeql-analysis.yml @@ -32,7 +32,7 @@ jobs: strategy: fail-fast: false matrix: - language: [ 'go', 'javascript' ] + language: [ 'go' ] # CodeQL supports [ 'cpp', 'csharp', 'go', 'java', 'javascript', 'python' ] # Learn more: # https://docs.github.com/en/free-pro-team@latest/github/finding-security-vulnerabilities-and-errors-in-your-code/configuring-code-scanning#changing-the-languages-that-are-analyzed diff --git a/.github/workflows/deploy-contract.yml b/.github/workflows/deploy-contract.yml index d3f32f61..bfe37f85 100644 --- a/.github/workflows/deploy-contract.yml +++ b/.github/workflows/deploy-contract.yml @@ -26,11 +26,11 @@ jobs: - uses: technote-space/get-diff-action@v6.0.1 id: git_diff with: - SUFFIX_FILTER: | - .go - .mod - .sum - .sol + PATTERNS: | + **/**.sol + **/**.go + go.mod + go.sum - name: Test contract run: | sudo make contract-tools diff --git a/.github/workflows/linkchecker.yml b/.github/workflows/linkchecker.yml deleted file mode 100644 index 8a381c51..00000000 --- a/.github/workflows/linkchecker.yml +++ /dev/null @@ -1,12 +0,0 @@ -name: Check Markdown links -on: - schedule: - - cron: "* */24 * * *" -jobs: - markdown-link-check: - runs-on: ubuntu-latest - steps: - - uses: actions/checkout@v3 - - uses: gaurav-nelson/github-action-markdown-link-check@1.0.14 - with: - folder-path: "docs" diff --git a/.github/workflows/lint.yml b/.github/workflows/lint.yml index aa190de5..5b79046a 100644 --- a/.github/workflows/lint.yml +++ b/.github/workflows/lint.yml @@ -18,7 +18,6 @@ jobs: go-version: 1.17 - uses: actions/checkout@v3 - uses: technote-space/get-diff-action@v6.0.1 - id: git_diff with: PATTERNS: | **/**.go @@ -31,3 +30,18 @@ jobs: github-token: ${{ secrets.github_token }} # Check only if there are differences in the source code if: "env.GIT_DIFF" + markdown-lint: + name: Run markdown-lint + runs-on: ubuntu-latest + timeout-minutes: 10 + steps: + - uses: actions/checkout@v3 + - uses: technote-space/get-diff-action@v6.0.1 + with: + PATTERNS: | + docs/**/*.md + x/**/*.md + README.md + - uses: articulate/actions-markdownlint@v1.1.0 + # Check only if there are differences in the source code + if: env.GIT_DIFF diff --git a/.github/workflows/semgrep.yml b/.github/workflows/semgrep.yml index b016762d..750a9da8 100644 --- a/.github/workflows/semgrep.yml +++ b/.github/workflows/semgrep.yml @@ -34,7 +34,7 @@ jobs: generateSarif: "1" if: "env.GIT_DIFF_FILTERED != ''" # Upload findings to GitHub Advanced Security Dashboard [step 2/2] - - name: Upload SARIF file for GitHub Advanced Security Dashboard + - name: Upload SARIF file uses: github/codeql-action/upload-sarif@v1 with: sarif_file: semgrep.sarif diff --git a/.github/workflows/test.yml b/.github/workflows/test.yml index 10500de7..f24791e1 100644 --- a/.github/workflows/test.yml +++ b/.github/workflows/test.yml @@ -43,7 +43,6 @@ jobs: - uses: technote-space/get-diff-action@v6.0.1 with: PATTERNS: | - **/**.sol **/**.go go.mod go.sum @@ -69,72 +68,14 @@ jobs: - uses: technote-space/get-diff-action@v6.0.1 id: git_diff with: - SUFFIX_FILTER: | - .go - .mod - .sum + PATTERNS: | + **/**.go + go.mod + go.sum - name: test-importer run: | make test-import if: env.GIT_DIFF - - test-solidity: - runs-on: ubuntu-latest - timeout-minutes: 240 - strategy: - fail-fast: false - matrix: - batch: ['1-3', '2-3', '3-3'] - steps: - - uses: actions/checkout@v3 - - uses: actions/setup-go@v3 - with: - go-version: 1.17 - check-latest: true - - uses: technote-space/get-diff-action@v6.0.1 - id: git_diff - with: - PATTERNS: | - **/**.sol - **/**.go - tests/solidity/**/*.js - tests/solidity/**/*.sh - go.mod - go.sum - - name: test-solidity - run: | - ./scripts/run-solidity-tests.sh --batch=${{ matrix.batch }} - if: env.GIT_DIFF - - liveness-test: - runs-on: ubuntu-latest - timeout-minutes: 10 - steps: - - uses: actions/checkout@v3 - - uses: actions/setup-go@v3 - with: - go-version: 1.17 - check-latest: true - - uses: technote-space/get-diff-action@v6.0.1 - id: git_diff - with: - PATTERNS: | - **/**.go - go.mod - go.sum - - name: Install Starport - run: | - curl https://get.starport.network/starport! | bash - if: env.GIT_DIFF - - name: Start Local Network via Starport - run: | - starport chain serve --reset-once -v -c ./starport.yml > starport.out 2>&1 & - if: env.GIT_DIFF - - name: Test Local Network Liveness - run: | - sleep 2m - ./contrib/scripts/test_localnet_liveness.sh 100 5 50 localhost - if: env.GIT_DIFF test-rpc: runs-on: ubuntu-latest