Skip to content

Commit 4062e71

Browse files
committed
ci(pipeline): update build evaluation steps in workflow
- Refactors build evaluation logic to improve clarity and maintainability. - Renames evaluation steps for build conditions to align with their functionalities. - Introduces dynamic build configuration options based on branch context, supporting better release management.
1 parent 92da42a commit 4062e71

File tree

1 file changed

+27
-13
lines changed

1 file changed

+27
-13
lines changed

.github/workflows/pipeline.yaml

Lines changed: 27 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -42,9 +42,10 @@ jobs:
4242
pathsFilter_src: ${{ steps.pathsFilter.outputs.src }}
4343
gitVersion_SemVer: ${{ steps.gitversion.outputs.GitVersion_SemVer }}
4444
gitVersion_AssemblySemFileVer: ${{ steps.gitversion.outputs.GitVersion_AssemblySemFileVer }}
45-
requiresBuild: ${{ steps.requires_build.outputs.result }}
46-
requiresBuildPush: ${{ steps.requires_build_push.outputs.result }}
47-
requiresRelease: ${{ steps.requires_release.outputs.result }}
45+
build: ${{ steps.evaluate_build.outputs.result }}
46+
build_push: ${{ steps.evaluate_build_push.outputs.result }}
47+
build_configuration: ${{ steps.evaluate_build_configuration.outputs.result }}
48+
release: ${{ steps.evaluate_release.outputs.result }}
4849
steps:
4950
- name: checkout
5051
uses: actions/checkout@v4
@@ -89,8 +90,8 @@ jobs:
8990
- 'playground/**'
9091
9192
92-
- name: evaluate - requires_build
93-
id: requires_build
93+
- name: evaluate - build
94+
id: evaluate_build
9495
run: |
9596
if [ "${{ steps.pathsFilter.outputs.build }}" = "true" ] || \
9697
[ "${{ github.event.inputs.force_build }}" = "true" ] || \
@@ -101,18 +102,28 @@ jobs:
101102
fi
102103
echo "result=true" >> $GITHUB_OUTPUT
103104
104-
- name: evaluate - requires_build_push
105-
id: requires_build_push
105+
- name: evaluate - build_push
106+
id: evaluate_build_push
106107
run: |
107-
if [ "${{ steps.requires_build.outputs.result }}" = "true" ]; then
108+
if [ "${{ steps.evaluate_build.outputs.result }}" = "true" ]; then
108109
result=true
109110
else
110111
result=false
111112
fi
112113
echo "result=$result" >> $GITHUB_OUTPUT
113114
114-
- name: evaluate - requires_release
115-
id: requires_release
115+
- name: evaluate - build_configuration
116+
id: evaluate_build_configuration
117+
run: |
118+
if [ "${{ github.ref }}" = "refs/heads/main" ]; then
119+
result=Release
120+
else
121+
result=Debug
122+
fi
123+
echo "result=$result" >> $GITHUB_OUTPUT
124+
125+
- name: evaluate - release
126+
id: evaluate_release
116127
run: |
117128
if [ "${{ github.ref }}" = "refs/heads/main" ] || \
118129
[ "${{ github.event.inputs.force_publish }}" = "true" ]; then
@@ -126,10 +137,13 @@ jobs:
126137
127138
build:
128139
name: build
129-
if: ${{ needs.discovery.outputs.requiresBuild == 'true' }}
140+
if: ${{ needs.discovery.outputs.build == 'true' }}
130141
needs: [discovery]
131142
runs-on: ubuntu-latest
132143
env:
144+
build: ${{ needs.discovery.outputs.build }}
145+
build_push: ${{ needs.discovery.outputs.build_push }}
146+
build_configuration: ${{ needs.discovery.outputs.build_configuration }}
133147
gitVersion_SemVer: ${{ needs.discovery.outputs.gitVersion_SemVer }}
134148
gitVersion_AssemblySemFileVer: ${{ needs.discovery.outputs.gitVersion_AssemblySemFileVer }}
135149
steps:
@@ -145,10 +159,10 @@ jobs:
145159
run: dotnet restore
146160

147161
- name: dotnet - build
148-
run: dotnet build --no-restore --configuration Release /p:Version=${{ env.gitVersion_SemVer }} /p:AssemblyVersion=${{env.gitVersion_AssemblySemFileVer}} /p:NuGetVersion=${{env.gitVersion_SemVer}}
162+
run: dotnet build --no-restore --configuration ${{ env.build_configuration }} /p:Version=${{ env.gitVersion_SemVer }} /p:AssemblyVersion=${{env.gitVersion_AssemblySemFileVer}} /p:NuGetVersion=${{env.gitVersion_SemVer}}
149163

150164
- name: dotnet - test
151-
run: dotnet test --no-build --configuration Release --verbosity normal
165+
run: dotnet test --no-build --configuration ${{ env.build_configuration }} --verbosity normal
152166

153167
- name: tests - report
154168
uses: dorny/test-reporter@v2

0 commit comments

Comments
 (0)