Skip to content

Commit

Permalink
ci: change build workflow (#346)
Browse files Browse the repository at this point in the history
* ci: fix build workflow

Correct the msbuild cli option.

References: #332

* ci: fix build config

References: #332

* ci: change the name of the 'build' job

References: #332
  • Loading branch information
al-kau authored Sep 24, 2024
1 parent cfad9e5 commit 5459add
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 6 deletions.
7 changes: 3 additions & 4 deletions .github/configurations/build.json
Original file line number Diff line number Diff line change
Expand Up @@ -133,14 +133,14 @@
"os": "windows-latest",
"project": "./source/Eppie.App/Eppie.App/Eppie.App.csproj",
"prerequisites": {
"msbuild": "true",
"msbuild": "false",
"uno-platform": "true"
},
"build": {
"tool": "msbuild",
"tool": "dotnet",
"configuration": "release",
"framework": "net8.0-windows10.0.19041",
"extra": "-verbosity:minimal -consoleLoggerParameters:Summary"
"extra": "--verbosity=minimal --nologo"
}
},

Expand Down Expand Up @@ -263,7 +263,6 @@
"build": {
"tool": "msbuild",
"configuration": "release",
"framework": "uap10.0.19041",
"extra": "-verbosity:minimal -consoleLoggerParameters:Summary"
}
},
Expand Down
4 changes: 2 additions & 2 deletions .github/workflows/build.yml
Original file line number Diff line number Diff line change
Expand Up @@ -99,7 +99,7 @@ jobs:
fi
build:
name: ${{ matrix.os }} ${{ matrix.build.framework }} '${{ matrix.project }}'
name: ${{ matrix.build.framework }} ${{ matrix.os }} '${{ matrix.project }}'
needs: prepare
strategy:
fail-fast: true
Expand Down Expand Up @@ -153,6 +153,6 @@ jobs:
$configuration='${{ matrix.build.configuration }}'
$extra_options=$('${{ matrix.build.extra }}' -split '\s+')
$framework='${{ matrix.build.framework }}'
$framework_option=$( if($framework) {"-property:TargetFramwork=$framework"} else {""} )
$framework_option=$( if($framework) {"-property:TargetFramework=$framework"} else {""} )
msbuild -target:Rebuild -restore:True -property:Configuration=$configuration $framework_option $extra_options $project

0 comments on commit 5459add

Please sign in to comment.