Skip to content

Commit

Permalink
cleanup: migrate make verify to static checks and remove redundant
Browse files Browse the repository at this point in the history
scripts

Signed-off-by: Mayank Shah <[email protected]>
  • Loading branch information
mayankshah1607 committed Oct 1, 2020
1 parent 341d133 commit 961a981
Show file tree
Hide file tree
Showing 7 changed files with 32 additions and 153 deletions.
45 changes: 32 additions & 13 deletions .github/workflows/static.yaml
Original file line number Diff line number Diff line change
@@ -1,16 +1,35 @@
name: Static Checks
on:
pull_request: {}
push: {}
push:
branches: [ master ]
pull_request:
branches: [ master ]
jobs:
go_lint:
name: Go Lint
runs-on: ubuntu-18.04
steps:
- name: Checkout code
uses: actions/checkout@master
- name: Run linter
uses: golangci/golangci-lint-action@v1
with:
version: v1.29
args: -E=gofmt --timeout=30m0s
static_checks:
name: Go Lint
runs-on: ubuntu-18.04
steps:
- name: Checkout code
uses: actions/checkout@master
- name: Run linter
uses: golangci/golangci-lint-action@v1
with:
version: v1.29
args: '-E=gofmt,golint,misspell --timeout=30m0s'
yaml_lint:
name: YAML lint
runs-on: ubuntu-18.04
steps:
- name: Checkout code
uses: actions/checkout@master
- name: yaml-lint
uses: ibiqlik/action-yamllint@v1
misc_checks:
name: Miscellaneous checks
runs-on: ubuntu-18.04
steps:
- name: Checkout code
uses: actions/checkout@master
- name: yaml-lint
run: |
make verify
1 change: 0 additions & 1 deletion .github/workflows/ubuntu.yml
Original file line number Diff line number Diff line change
Expand Up @@ -30,7 +30,6 @@ jobs:
- name: Build Test
run: |
export PATH=$PATH:$HOME/.local/bin
make verify
go test -covermode=count -coverprofile=profile.cov ./pkg/...
export DOCKER_CLI_EXPERIMENTAL=enabled && make container
Expand Down
5 changes: 0 additions & 5 deletions hack/verify-all.sh
Original file line number Diff line number Diff line change
Expand Up @@ -18,11 +18,6 @@ set -euo pipefail

readonly PKG_ROOT="$(git rev-parse --show-toplevel)"

${PKG_ROOT}/hack/verify-gofmt.sh
${PKG_ROOT}/hack/verify-govet.sh
${PKG_ROOT}/hack/verify-golint.sh
${PKG_ROOT}/hack/verify-yamllint.sh
${PKG_ROOT}/hack/verify-boilerplate.sh
${PKG_ROOT}/hack/verify-spelling.sh
${PKG_ROOT}/hack/verify-helm-chart.sh
${PKG_ROOT}/hack/verify-gomod.sh
28 changes: 0 additions & 28 deletions hack/verify-gofmt.sh

This file was deleted.

26 changes: 0 additions & 26 deletions hack/verify-golint.sh

This file was deleted.

23 changes: 0 additions & 23 deletions hack/verify-govet.sh

This file was deleted.

57 changes: 0 additions & 57 deletions hack/verify-spelling.sh

This file was deleted.

0 comments on commit 961a981

Please sign in to comment.