Skip to content

ci+docs: CI matrix & coverage badges, NuGet release badges#3

Merged
monsieurleberre merged 1 commit into
mainfrom
docs/ci-badges-nuget
Jun 15, 2026
Merged

ci+docs: CI matrix & coverage badges, NuGet release badges#3
monsieurleberre merged 1 commit into
mainfrom
docs/ci-badges-nuget

Conversation

@monsieurleberre

Copy link
Copy Markdown
Contributor

Mirrors the badge work from daml-codegen-csharp#9, adapted for this single-language repo.

CI workflow (.github/workflows/ci.yaml)

  • Add artifact-prefix: csharp to the csharp-ci call so per-run artifacts (coverage value, shard results) are namespaced.
  • Add an assemble-badge-data job that tags the csharp-ci matrix-status output with lang: "csharp".
  • Add a badges job (on main pushes only, contents: write) calling the shared update-badges.yaml@v2 reusable workflow to publish per-shard CI badges and the coverage badge to the orphan badges branch.

README

  • Replace the stale static Release-v0.1.4 badge (and its now-incorrect "repo is private" comment) with dynamic latest stable / latest preview release badges — the repo is public now, so shields.io's github/v/release/... endpoint resolves.
  • Add a CI workflow-status badge.
  • Add a Platform Support matrix: one badge per OS × arch shard (Ubuntu / Windows / macOS × amd64 / arm64 — all six are supported here, including windows-11-arm).
  • Add a Coverage badge.
  • Add latest stable / latest preview release badges to the Packages section.

No per-language sub-sections — this repo ships only C#, so the badge groups are flat.

Note

The shields endpoint badges (matrix + coverage) read from the badges orphan branch, which is created/populated on the first main run after this merges. Until then they render as "invalid" — expected.

Copilot AI review requested due to automatic review settings June 15, 2026 08:25

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@github-actions

Copy link
Copy Markdown
Contributor

C# coverage

Code Coverage

Package Line Rate Branch Rate Complexity Health
Canton.Ledger.Auth 100% 98% 62
Canton.Ledger.Grpc.Client 97% 83% 190
Canton.Ledger.Pqs.Client 84% 88% 54
Daml.Runtime.Grpc 99% 99% 82
Summary 96% (924 / 966) 89% (327 / 366) 388

@monsieurleberre monsieurleberre merged commit 9ad37a8 into main Jun 15, 2026
13 checks passed
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.

2 participants