Skip to content

Port copilot-instructions.md changes from main#37763

Closed
Copilot wants to merge 2 commits intorelease/10.0from
copilot/port-copilot-instructions-changes
Closed

Port copilot-instructions.md changes from main#37763
Copilot wants to merge 2 commits intorelease/10.0from
copilot/port-copilot-instructions-changes

Conversation

Copy link
Contributor

Copilot AI commented Feb 21, 2026

Ports the .github/copilot-instructions.md updates from PR #37636 into this branch.

Changes

  • Comments – Replaced vague "add rarely" with explicit "Prefer minimal comments" guideline clarifying why vs what
  • Pattern matching – Added concrete before/after example for is over as + null check; added preference for when clauses in switch expressions
  • Testing – Added cross-platform verification note and manual dotnet-ef test guidance
  • New section: Dependency and Version Management – Never hardcode package versions; use eng/Versions.props, Directory.Packages.props; don't touch Maestro-managed packages in eng/Version.Details.xml
  • Implementation Guidelines – Added note to review surrounding code for similar patterns after fixing
  • New section: Pull Request Guidelines – Always target main; includes servicing PR description template
Original prompt

Port the copilot-instructions.md‎ file changes


🔒 GitHub Advanced Security automatically protects Copilot coding agent pull requests. You can protect all pull requests by enabling Advanced Security for your repositories. Learn more about Advanced Security.

Copilot AI self-assigned this Feb 21, 2026
Co-authored-by: AndriySvyryd <6539701+AndriySvyryd@users.noreply.github.com>
Copilot AI changed the title [WIP] Port changes from copilot-instructions.md file Port copilot-instructions.md changes from main Feb 21, 2026
Copilot AI requested a review from AndriySvyryd February 21, 2026 02:30
@AndriySvyryd AndriySvyryd deleted the copilot/port-copilot-instructions-changes branch February 26, 2026 01:47
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