Skip to content

Strengthen contributor guidance to support Help-Wanted contributions - #6402

Open
Kaleb Luedtke (Trenly) wants to merge 6 commits into
microsoft:masterfrom
Trenly:ContributorDocs
Open

Strengthen contributor guidance to support Help-Wanted contributions#6402
Kaleb Luedtke (Trenly) wants to merge 6 commits into
microsoft:masterfrom
Trenly:ContributorDocs

Conversation

@Trenly

@Trenly Kaleb Luedtke (Trenly) commented Jul 29, 2026

Copy link
Copy Markdown
Contributor

📖 Description

Strengthens contributor guidance to better support Help-Wanted contributions. Addresses the four sub-issues tracked under #6393:

  • Adds an explicit testing policy to CONTRIBUTING.md specifying when unit and E2E tests are expected, when they may be omitted, and how to run them.
  • Adds doc/Standards.md, a new coding conventions reference covering file formatting, brace style, naming (C++ and C#), error handling (WIL/AICLI_* macros), casts, std::move() usage, and resource string requirements.
  • Adds an AI-assisted contributions policy to CONTRIBUTING.md clarifying that contributors are fully accountable for AI-generated code and must disclose material AI use in their PR.
  • Updates .github/PULL_REQUEST_TEMPLATE.md to add an evidence prompt in the Validation section, a tests checkbox, and an AI Assistance disclosure section.
  • Updates doc/Developing.md to cross-reference doc/Standards.md and add a dedicated "Running End-to-End Tests" section.
  • Updates .github/copilot-instructions.md to document PR expectations for AI-assisted work and normalize trailing whitespace.

cc Demitrius Nelon (@denelon)

🔗 References

🔍 Validation

Documentation-only changes. All cross-references and Markdown anchor links verified. No functional code changed.

✅ Checklist

🤖 AI Assistance

  • AI assistance was used and has been disclosed in this PR

GitHub Copilot assisted in authoring all documentation additions in this PR: the new doc/Standards.md file, the AI-assisted contributions policy, the expanded testing guidance in CONTRIBUTING.md, the "Running End-to-End Tests" section in doc/Developing.md, and the PR template updates. I have reviewed and revised the submitted content.

📋 Issue Type

  • Bug fix
  • Feature
  • Task
Microsoft Reviewers: Open in CodeFlow

Kaleb Luedtke (Trenly) and others added 5 commits July 28, 2026 20:31
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
* Add screenshot/recording evidence prompt to Validation section
* Add 'AI assistance was used and has been disclosed' checkbox
* Add 'No AI assistance was used' checkbox
* Add 'Added or updated tests (or noted why not applicable)' checkbox

Resolves microsoft#6397

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines:
There may be pipelines that require an authorized user to comment /azp run to run.

@github-actions

This comment has been minimized.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@Trenly
Kaleb Luedtke (Trenly) marked this pull request as ready for review August 6, 2026 02:56
@Trenly
Kaleb Luedtke (Trenly) requested a review from a team as a code owner August 6, 2026 02:56
@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines:
There may be pipelines that require an authorized user to comment /azp run to run.

@JohnMcPMS

Copy link
Copy Markdown
Member

I would want Demitrius Nelon (@denelon) to look at this as well.

I don't think we need to have PRs explicitly state the AI usage in the template.

Comment thread doc/Standards.md
// Copyright (c) Microsoft Corporation. Licensed under the MIT License.
// </copyright>
// -----------------------------------------------------------------------------
```

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

We should only add Microsoft copyright headers to files coded by Microsoft employees.

Contributors can add their own copyright header to their original contributions, as well as to files they modify and contribute back.

@florelis

Copy link
Copy Markdown
Member

I don't think we need to have PRs explicitly state the AI usage in the template.

I don't care much if something was made with AI assistance, but I'd like to know if something is fully AI generated. Especially if the submitter didn't review it or it includes things they don't fully understand.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Needs-Author-Feedback Issue needs attention from issue or PR author

Projects

None yet

3 participants