Skip to content

Commit

Permalink
fixes
Browse files Browse the repository at this point in the history
  • Loading branch information
externl committed Dec 4, 2024
1 parent 2d4a1bc commit 010c66d
Show file tree
Hide file tree
Showing 3 changed files with 6 additions and 6 deletions.
7 changes: 4 additions & 3 deletions .github/workflows/coverage.yml
Original file line number Diff line number Diff line change
Expand Up @@ -64,9 +64,10 @@ jobs:
with:
use_ccache: false

- name: Build C#
working-directory: ./csharp
run: make
- name: Build
run: |
make -C cpp srcs
make -C csharp
- name: Generate C# Coverage
working-directory: ./csharp
Expand Down
1 change: 1 addition & 0 deletions csharp/msbuild/coverage.runsettings
Original file line number Diff line number Diff line change
Expand Up @@ -3,6 +3,7 @@
<DataCollectionRunSettings>
<DataCollectors>
<DataCollector friendlyName="Code Coverage">
<Format>Cobertura</Format>
<Configuration>
<CodeCoverage>
<ModulePaths>
Expand Down
4 changes: 1 addition & 3 deletions scripts/Util.py
Original file line number Diff line number Diff line change
Expand Up @@ -3950,9 +3950,7 @@ def getCommandLine(self, current, process, exe, args):
if current.config.dotnetCoverageSession != "":
# escapign \" is requried for passing though from dotnet-coverage -> dotnet -> exe
cmd = cmd.replace('\\"', '\\\\\\"')
cmd = (
f"dotnet-coverage connect {current.config.dotnetCoverageSession} {cmd}"
)
cmd = f"dotnet-coverage connect --nologo {current.config.dotnetCoverageSession} {cmd}"

return cmd

Expand Down

0 comments on commit 010c66d

Please sign in to comment.