Skip to content

Make .NET portability workflow a weekly audit - #1173

Merged
Quim Muntal (qmuntal) merged 1 commit into
mainfrom
weekly-dotnet-portability-audit
Sep 23, 2026
Merged

Quim Muntal (qmuntal) merged 1 commit into
mainfrom
weekly-dotnet-portability-audit

Conversation

@qmuntal

Copy link
Copy Markdown
Member

Summary

Replace the nightly refactoring quota with a weekly portability audit on Mondays at 22:19 UTC. Keep manual dispatch and the [dotnet-code] output prefix.

  • Require a concrete porting obstacle or useful missing behavioral test, pinned upstream evidence, and focused behavior-preservation coverage. Treat noop as a successful result instead of producing cosmetic changes.
  • Check open and closed tracking issues, PRs, and maintainer feedback; retain a bounded audit history. Leave bug fixes and feature ports to the companion workflows.
  • Fetch and validate upstream .NET source before the sandboxed agent starts, pin the inspected commit, and fail setup when upstream is unavailable without widening sandbox network access.
  • Rename the workflow and tracker to weekly, restrict proposals to SDK directories, and regenerate the lock workflow with gh-aw v0.88.7.

Validation

  • Strict gh-aw compilation and validation.
  • actionlint with narrow compatibility exceptions for the existing copilot-requests permission and concurrency.queue fields.
  • Bash syntax and generated setup ordering checks.
  • Five mocked setup scenarios: success, failed fetch, missing ref, missing source tree, and missing test tree.
  • Working-tree and staged diff whitespace checks.

No live agent run was dispatched.

Replace nightly refactoring with a weekly, evidence-backed audit.
Require concrete portability or test coverage benefits, check prior
issues and review outcomes, and allow no-op results.

Fetch and pin the upstream .NET reference before sandbox execution.
Rename the workflow and tracker, limit proposals to SDK directories,
and regenerate the workflow with gh-aw v0.88.7.
@qmuntal
Quim Muntal (qmuntal) requested a review from a team as a code owner September 23, 2026 12:15
Copilot AI balanced review requested due to automatic review settings September 23, 2026 12:15
@github-actions github-actions Bot added area:github Changes files in the github area size:xlarge More than 300 changed lines or 10 files labels Sep 23, 2026

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Copilot review overview

🟢 Approval recommended

The source and generated workflow consistently implement the stated schedule, validation, history, and output constraints.

Review effort: Balanced
Findings: None

What changed in this PR

Converts the nightly portability workflow into a weekly, evidence-driven audit.

Changes:

  • Schedules weekly Monday audits with optional no-change outcomes.
  • Adds pinned upstream validation and bounded audit history.
  • Restricts proposals to SDK directories and regenerates the workflow lock.
File Description
.github/​workflows/​dotnet-code-portability-weekly.md Defines the weekly audit.
.github/​workflows/​dotnet-code-portability-weekly.lock.yml Contains the regenerated executable workflow.
.github/​workflows/​dotnet-code-portability-nightly.md Removes the superseded nightly workflow.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

@github-actions github-actions Bot added the kind:ci Changes CI, build, or repository automation label Sep 23, 2026
@qmuntal
Quim Muntal (qmuntal) added this pull request to the merge queue Sep 23, 2026
Merged via the queue into main with commit 5fea526 Sep 23, 2026
22 checks passed
@qmuntal
Quim Muntal (qmuntal) deleted the weekly-dotnet-portability-audit branch September 23, 2026 12:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area:github Changes files in the github area kind:ci Changes CI, build, or repository automation size:xlarge More than 300 changed lines or 10 files

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants