Skip to content

Document that Python and .NET changes go in separate PRs - #9168

Closed
Mark Licata (marklicata) wants to merge 1 commit into
microsoft:mainfrom
marklicata:contributing-split-language-prs
Closed

Mark Licata (marklicata) wants to merge 1 commit into
microsoft:mainfrom
marklicata:contributing-split-language-prs

Conversation

@marklicata

@marklicata Mark Licata (marklicata) commented Oct 7, 2026 •

Copy link
Copy Markdown

Motivation & Context

Reviewers have asked that Python and .NET changes not be combined in one pull request, because cross-language PRs are much harder to review. That expectation isn't written down anywhere today. The only hint is the Python: / .NET: title-prefix convention. This documents it in the contribution guidelines.

Description & Review Guide

What are the major changes?

  • New DON'T in CONTRIBUTING.md: don't combine Python and .NET changes in a single PR
  • Step 7 of the suggested workflow now says to keep each PR to one language, use the Python: / .NET: title prefix, and link the PR in the other language

What is the impact of these changes?

Documentation only.

What do you want reviewers to focus on?

  • Whether the wording matches the team's intended policy

Related Issue

Closes #9170

Contribution Checklist

  • The code builds clean without any errors or warnings
  • All unit tests pass, and I have added new tests where possible
  • The PR follows the Contribution Guidelines
  • This PR is linked to an issue and there is no other open PR for this issue (see Related Issue above).
  • This is not a breaking change.

🤖 Generated with Claude Code

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>

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.

🟢 Approval recommended

The documentation clearly and consistently captures the intended contribution policy.

0 open findings

What changed in this PR

Documents the policy that Python and .NET changes should use separate pull requests.

Changes:

  • Adds the policy to the contribution guidelines.
  • Updates the recommended PR workflow with language prefixes and cross-linking guidance.
File Description
CONTRIBUTING.md Documents language-specific PR requirements.

🧠 Review effort: Balanced


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

@marklicata

Copy link
Copy Markdown
Author

Per Eduard van Valkenburg (@eavanvalkenburg) this change isn't needed. This will be folded into other work the core team is doing.

@marklicata

Copy link
Copy Markdown
Author

Per Eduard van Valkenburg (@eavanvalkenburg) this change isn't needed. This will be folded into other work the core team is doing.

@marklicata
Mark Licata (marklicata) deleted the contributing-split-language-prs branch October 7, 2026 17:09

This branch was previously deployed

1 inactive deployment
github-app-auth — e6bb75cc Deployed Oct 7, 2026 by marklicata via team_check #6288
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Usage: [Issues, PRs], Target: documentation in the code base and learn docs

Projects

None yet

Development

Successfully merging this pull request may close these issues.

.NET: Python: Document that Python and .NET changes go in separate PRs

2 participants