ci: dispatch coverage-fanout (not csharp-coverage) on merged C# PRs#590
Merged
Conversation
The driver-test coverage entry point moved from the csharp-only engineer-bot.yml
to coverage-fanout-tracker.yml (databricks-driver-test#857). Update the post-merge
sender to emit event-type `coverage-fanout` with a language-tagged payload
{reference_language: csharp, pr_number, pr_url} instead of `csharp-coverage`.
This drives the multi-language fan-out (reference: csharp) + issue tracking.
Hard swap: lands together with databricks-driver-test#857.
Co-authored-by: Isaac
eric-wang-1990
requested review from
gopalldb,
lidavidm,
msrathore-db and
vikrantpuppala
as code owners
July 20, 2026 22:15
eric-wang-1990
added a commit
that referenced
this pull request
Jul 21, 2026
…ispatch (#591) Follow-up to #590. The coverage fan-out went **language-free** (databricks-driver-test#857): the reference is a **repo + PR**, and spec authoring derives the source subtree from the PR's own changed files — no language token. This updates the post-merge sender payload: ``` - {"reference_language": "csharp", "pr_number": ..., "pr_url": ...} + {"reference_repo": "${{ github.repository }}", "pr_number": ..., "pr_url": ...} ``` `github.repository` is `adbc-drivers/databricks` here; the fan-out derives the csharp-vs-rust binding from which subtree the PR touched. No other change.⚠️ Lands **lockstep** with databricks-driver-test#857 (the tracker now expects `reference_repo`). This pull request and its description were written by Isaac.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
What
Post-merge C# coverage sender (
trigger-integration-tests.yml→trigger-csharp-coverage) now dispatchescoverage-fanouttodatabricks-driver-testwith a language-tagged payload:{"reference_language": "csharp", "pr_number": "...", "pr_url": "..."}instead of the old
csharp-coverageevent-type.Why
The coverage entry point in
databricks-driver-testmoved from the csharp-onlyengineer-bot.yml(being deleted) to the newcoverage-fanout-tracker.yml, which creates a tracking issue and drives the multi-language fan-out (reference: csharp). See databricks-driver-test#857.The tracker listens only on
coverage-fanout(no back-compat forcsharp-coverage). This PR and databricks-driver-test#857 must merge together, or post-merge C# coverage pauses until both are in. No behavior change for any other job in this workflow.This pull request and its description were written by Isaac.