Skip to content

Commit e064a93

Browse files
committed
Refactor CI workflow: update integration test steps and remove unused linting commands
1 parent 9386a54 commit e064a93

File tree

1 file changed

+37
-28
lines changed

1 file changed

+37
-28
lines changed

.github/workflows/ci-build.yml

Lines changed: 37 additions & 28 deletions
Original file line numberDiff line numberDiff line change
@@ -37,39 +37,48 @@ jobs:
3737
with:
3838
go-version: "^1.24.0"
3939

40-
- name: "Install extra tools"
41-
run: |
42-
curl -sSfL https://raw.githubusercontent.com/golangci/golangci-lint/master/install.sh | sh -s -- -b $(go env GOPATH)/bin
40+
- name: "Dapr tool installer"
41+
uses: dapr/setup-dapr@v1
4342

44-
- name: "Check code & lint"
43+
- name: "Run integration tests"
4544
run: |
46-
go get ./...
47-
make lint
45+
make run &
46+
sleep 30
47+
make test-api
48+
49+
# - name: "Install extra tools"
50+
# run: |
51+
# curl -sSfL https://raw.githubusercontent.com/golangci/golangci-lint/master/install.sh | sh -s -- -b $(go env GOPATH)/bin
4852

49-
- name: "Run all unit tests"
50-
run: make test
53+
# - name: "Check code & lint"
54+
# run: |
55+
# go get ./...
56+
# make lint
57+
58+
# - name: "Run all unit tests"
59+
# run: make test
5160

5261
# ===== Build container images ======
53-
build-images:
54-
runs-on: ubuntu-latest
55-
needs: tests-linting
56-
strategy:
57-
matrix:
58-
serviceName: [cart, orders, users, products, frontend]
62+
# build-images:
63+
# runs-on: ubuntu-latest
64+
# needs: tests-linting
65+
# strategy:
66+
# matrix:
67+
# serviceName: [cart, orders, users, products, frontend]
5968

60-
steps:
61-
- name: "Checkout source"
62-
uses: actions/checkout@v3
69+
# steps:
70+
# - name: "Checkout source"
71+
# uses: actions/checkout@v3
6372

64-
- name: "Run build"
65-
run: |
66-
make docker-build-${{ matrix.serviceName }} IMAGE_TAG=$IMAGE_TAG VERSION=$VERSION BUILD_INFO="$BUILD_INFO"
67-
make docker-build-${{ matrix.serviceName }} IMAGE_TAG=latest VERSION=$VERSION BUILD_INFO="$BUILD_INFO"
73+
# - name: "Run build"
74+
# run: |
75+
# make docker-build-${{ matrix.serviceName }} IMAGE_TAG=$IMAGE_TAG VERSION=$VERSION BUILD_INFO="$BUILD_INFO"
76+
# make docker-build-${{ matrix.serviceName }} IMAGE_TAG=latest VERSION=$VERSION BUILD_INFO="$BUILD_INFO"
6877

69-
# Steps after this are only run when merging or pushing into main
70-
- name: "Push to container registry"
71-
if: github.event_name == 'push' && github.ref == 'refs/heads/main'
72-
run: |
73-
echo ${{ secrets.GITHUB_TOKEN }} | docker login $IMAGE_REG -u $GITHUB_ACTOR --password-stdin
74-
make docker-push-${{ matrix.serviceName }} IMAGE_TAG=$IMAGE_TAG
75-
make docker-push-${{ matrix.serviceName }} IMAGE_TAG=latest
78+
# # Steps after this are only run when merging or pushing into main
79+
# - name: "Push to container registry"
80+
# if: github.event_name == 'push' && github.ref == 'refs/heads/main'
81+
# run: |
82+
# echo ${{ secrets.GITHUB_TOKEN }} | docker login $IMAGE_REG -u $GITHUB_ACTOR --password-stdin
83+
# make docker-push-${{ matrix.serviceName }} IMAGE_TAG=$IMAGE_TAG
84+
# make docker-push-${{ matrix.serviceName }} IMAGE_TAG=latest

0 commit comments

Comments
 (0)