Skip to content

ci: fail the static code analysis job when the Sonar quality gate fails on pull requests - #210

Merged
vbreuss merged 1 commit into
mainfrom
ci/sonar-quality-gate-in-job
Nov 3, 2026
Merged

vbreuss merged 1 commit into
mainfrom
ci/sonar-quality-gate-in-job

Conversation

@vbreuss

@vbreuss vbreuss commented Nov 3, 2026

Copy link
Copy Markdown
Member

The SonarCloud status cannot be a required check, because it is never reported for pull requests from forks or Dependabot, which have no access to the Sonar token. When the job itself fails on the quality gate, the job can be the required check instead: it is skipped for those pull requests, which GitHub counts as passed.

…ls on pull requests

The SonarCloud status cannot be a required check, because it is never reported for pull requests from forks or Dependabot, which have no access to the Sonar token. When the job itself fails on the quality gate, the job can be the required check instead: it is skipped for those pull requests, which GitHub counts as passed.
@vbreuss vbreuss self-assigned this Nov 3, 2026
@vbreuss
vbreuss enabled auto-merge (squash) November 3, 2026 00:04
@sonarqubecloud

sonarqubecloud Bot commented Nov 3, 2026

Copy link
Copy Markdown

@github-actions

github-actions Bot commented Nov 3, 2026

Copy link
Copy Markdown

Test Results

    8 files  ±0      8 suites  ±0   1m 29s ⏱️ -2s
  454 tests ±0    454 ✅ ±0  0 💤 ±0  0 ❌ ±0 
2 961 runs  ±0  2 961 ✅ ±0  0 💤 ±0  0 ❌ ±0 

Results for commit a6372a1. ± Comparison against base commit 8090d0d.

This pull request removes 4 and adds 1 tests. Note that renamed tests count towards both.
  "bar": 2,
  "foo": 1,
aweXpect.Mockolate.Tests.Web.ItExtensionsTests+HttpContentTests+IsJsonContentTests ‑ WithBody_ShouldCompareAsJson({
}, {"bar":2,"foo": 1}, True)
aweXpect.Mockolate.Tests.Web.ItExtensionsTests+HttpContentTests+IsJsonContentTests ‑ WithBody_ShouldCompareAsJson({
  "foo": 1,
  "bar": 2,
}, {"bar":2,"foo": 1}, True)

@vbreuss
vbreuss merged commit 08e17fe into main Nov 3, 2026
16 checks passed
@vbreuss
vbreuss deleted the ci/sonar-quality-gate-in-job branch November 3, 2026 00:41
@github-actions

github-actions Bot commented Nov 3, 2026

Copy link
Copy Markdown

🚀 Benchmark Results

Details

BenchmarkDotNet v0.15.8, Linux Ubuntu 26.04.1 LTS (Resolute Raccoon)
AMD EPYC 9V45 4.02GHz, 1 CPU, 4 logical and 2 physical cores
.NET SDK 10.0.303
[Host] : .NET 10.0.14 (10.0.14, 10.0.1426.52326), X64 RyuJIT x86-64-v4
DefaultJob : .NET 10.0.14 (10.0.14, 10.0.1426.52326), X64 RyuJIT x86-64-v4

Method Mean Error StdDev Median Allocated
Dummy_aweXpect 0.0227 ns 0.0229 ns 0.0225 ns 0.0133 ns -

@github-actions

github-actions Bot commented Nov 3, 2026

Copy link
Copy Markdown

👽 Mutation Results

Mutation testing badge

aweXpect.Mockolate

Details
File Score Killed Survived Timeout No Coverage Ignored Compile Errors Runtime Errors Total Detected Total Undetected Total Mutants

The final mutation score is NaN%

Coverage Thresholds: high:80 low:60 break:0

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant