Skip to content

Commit 41c9b34

Browse files
ci: rewrite workflow with corrected YAML
Rewrote .github/workflows/ci-pipeline.yml for clarity and correctness. - Fixed on: block indentation (push + workflow_dispatch) - Ensured concurrency is properly scoped - Indented steps consistently under each job - Docker build now accepts BUILD_ID and produces tagged image - Both Linux and Windows artifacts upload correctly
1 parent 3ea9407 commit 41c9b34

1 file changed

Lines changed: 12 additions & 14 deletions

File tree

.github/workflows/ci-pipeline.yml

Lines changed: 12 additions & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -9,7 +9,6 @@ concurrency:
99
group: ${{ github.workflow }}-${{ github.ref }}
1010
cancel-in-progress: true
1111

12-
1312
jobs:
1413
build-linux:
1514
name: Build on Linux
@@ -23,20 +22,19 @@ jobs:
2322

2423
- name: Set up Docker Buildx
2524
uses: docker/setup-buildx-action@v3
26-
27-
- name: Build sample image
28-
run: |
29-
docker build \
30-
--build-arg BUILD_ID=${{ github.run_id }} \
31-
-t myorg/sample-app:ci-${{ github.run_id }} \
32-
./exercises/02-containerize-app/app
33-
3425

35-
- name: Upload linux artifact
36-
uses: actions/upload-artifact@v4
37-
with:
38-
name: linux-app-source
39-
path: ./exercises/02-containerize-app/app
26+
- name: Build sample image
27+
run: |
28+
docker build \
29+
--build-arg BUILD_ID=${{ github.run_id }} \
30+
-t myorg/sample-app:ci-${{ github.run_id }} \
31+
./exercises/02-containerize-app/app
32+
33+
- name: Upload linux artifact
34+
uses: actions/upload-artifact@v4
35+
with:
36+
name: linux-app-source
37+
path: ./exercises/02-containerize-app/app
4038

4139
build-windows:
4240
name: Build on Windows

0 commit comments

Comments
 (0)