Skip to content

ci: live coverage badge via update-badges workflow#9

Merged
monsieurleberre merged 11 commits into
mainfrom
feat/coverage-badge
Jun 15, 2026
Merged

ci: live coverage badge via update-badges workflow#9
monsieurleberre merged 11 commits into
mainfrom
feat/coverage-badge

Conversation

@monsieurleberre

Copy link
Copy Markdown
Contributor

Pilot for peacefulstudio/github-actions#23 (issue #22).

What

  • New post-CI badges job: needs: csharp-ci, main-only, permissions: contents: write. Calls the update-badges reusable workflow with coverage: ${{ needs.csharp-ci.outputs.coverage }}, which writes a shields.io endpoint JSON to an orphan badges branch via the built-in GITHUB_TOKEN (no gist, no PAT).
  • README: live coverage badge pointing at …/badges/coverage.json.

⚠️ Temporary ref — repoint before merge

csharp-ci and update-badges are pinned to the feat/coverage-badges PR branch of github-actions so this tests end-to-end. Do not merge as-is. Once github-actions#23 merges and v2 advances:

  1. Repoint both @feat/coverage-badges@v2 in .github/workflows/ci.yaml.
  2. The badge renders after the first main run writes the badges branch.

Not included

publish-splice.yaml has an unrelated pre-existing local edit; left out of this branch deliberately.

Wire a post-CI `badges` job that publishes a live coverage badge to an
orphan `badges` branch, and reference it from the README.

The `csharp-ci` and `update-badges` reusable workflows are temporarily
pinned to the `feat/coverage-badges` PR branch of peacefulstudio/github-actions
so this can be tested end-to-end. Repoint both to `@v2` once that PR is
merged and the v2 tag is advanced.
Copilot AI review requested due to automatic review settings June 14, 2026 09:47

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.

Copilot AI review requested due to automatic review settings June 14, 2026 09:57

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.

Copilot AI review requested due to automatic review settings June 14, 2026 12:35

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.

- Pass `artifact-prefix: csharp`/`scala` to the reusable CI workflows so the
  two languages no longer collide on run-level artifact names.
- Render Platform Support as 3-column grids (Ubuntu/Windows/macOS, amd64 over
  arm64); Scala's Windows-arm64 cell stays empty.
- Surface the latest-preview release badge in the top block.
…in-arm64 unsupported

- Drop the duplicate coverage badge from the top block (the dedicated
  Coverage section already shows it).
- Fill the empty Scala Windows/arm64 cell with a grey "not supported" badge.
Copilot AI review requested due to automatic review settings June 14, 2026 13:15

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.

Reverts the temporary feat/coverage-badge un-gate used for live badge
testing; badge writes are main-gated again ahead of merge.
The badges feature shipped in github-actions v2.2.0, so pin back to the
floating @v2 tag instead of the @feat/coverage-badges branch.
Copilot AI review requested due to automatic review settings June 14, 2026 13:57

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

Scala coverage

Code Coverage

Package Line Rate Branch Rate Health
studio.peaceful.daml.codegen.helper 92% 91%
Summary 92% (503 / 549) 91% (106 / 116)

@github-actions

Copy link
Copy Markdown
Contributor

C# coverage

Code Coverage

Package Line Rate Branch Rate Complexity Health
Daml.Codegen.CSharp 97% 96% 929
Daml.Codegen.CSharp.Cli 100% 100% 2
Daml.Codegen.Testing.Conformance 97% 97% 50
Daml.Ledger.Abstractions 95% 90% 12
Daml.Runtime 96% 90% 396
Summary 97% (3252 / 3351) 94% (1231 / 1305) 1389

@monsieurleberre monsieurleberre merged commit 7a9e7de into main Jun 15, 2026
22 checks passed
@monsieurleberre monsieurleberre deleted the feat/coverage-badge branch June 15, 2026 06:16
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.

3 participants