Skip to content

Commit

Permalink
workflow change: Separate lint from test workflow (#463)
Browse files Browse the repository at this point in the history
* remove old scripts and add submodule for upload scripts

* separate lint from test.yml

* fix test.yml

* update gitmodules

* remove old git submodule
  • Loading branch information
ykim-1 committed Mar 5, 2024
1 parent 3de1f50 commit a64312e
Show file tree
Hide file tree
Showing 3 changed files with 24 additions and 13 deletions.
23 changes: 23 additions & 0 deletions .github/workflows/pull_request.yml
Original file line number Diff line number Diff line change
@@ -0,0 +1,23 @@
name: Pull Request
on: pull_request
jobs:
tests:
runs-on: ubuntu-latest
steps:
- uses: actions/checkout@v4
- uses: actions/setup-go@v5
with:
go-version: 'stable'
- run: go version

- name: Run tidy
run: make tidy

- name: Run fmt
run: go fmt ./...

- name: Run linter
run: make lint

- name: Fail if changes
run: git diff-index --exit-code HEAD
12 changes: 0 additions & 12 deletions .github/workflows/test.yml
Original file line number Diff line number Diff line change
Expand Up @@ -25,18 +25,6 @@ jobs:
go-version: 'stable'
- run: go version

- name: Run tidy
run: make tidy

- name: Run fmt
run: go fmt ./...

- name: Fail if changes
run: git diff-index --exit-code HEAD

- name: Run linter
run: make lint

- name: Update system packages
run: sudo apt-get update -y

Expand Down
2 changes: 1 addition & 1 deletion tod_scripts

0 comments on commit a64312e

Please sign in to comment.