Skip to content

Commit 4a46973

Browse files
committed
ci: run Makefile targets
Rather than invoke 'go' directly, call the appropriate Makefile targets for the given operations in 'main.yml'. This more accurately reflects the build process for users and CD and avoids having multiple "sources of truth" on how to build the project. Signed-off-by: Victoria Dye <[email protected]>
1 parent ec543e7 commit 4a46973

File tree

1 file changed

+3
-11
lines changed

1 file changed

+3
-11
lines changed

.github/workflows/main.yml

Lines changed: 3 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -16,19 +16,11 @@ jobs:
1616
with:
1717
go-version-file: 'src/git-bundle-server/go.mod'
1818

19-
# TODO: when the '-C' option is available, remove the 'cd ...'
20-
# See https://github.com/golang/go/issues/50332 for more details
2119
- name: Build
22-
run: |
23-
cd $GITHUB_WORKSPACE/src/git-bundle-server
24-
go build -v ./...
20+
run: make -C src/git-bundle-server build
2521

2622
- name: Check style
27-
run: |
28-
cd $GITHUB_WORKSPACE/src/git-bundle-server
29-
go vet ./...
23+
run: make -C src/git-bundle-server vet
3024

3125
- name: Run unit tests
32-
run: |
33-
cd $GITHUB_WORKSPACE/src/git-bundle-server
34-
go test -v ./...
26+
run: make -C src/git-bundle-server test

0 commit comments

Comments
 (0)