Skip to content

Strengthen contributor guidance to support Help-Wanted contributions #6402

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
Demitrius Nelon (denelon) merged 9 commits into from
Sep 22, 2026
Merged

Strengthen contributor guidance to support Help-Wanted contributions #6402

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
9 commits
Select commit Hold shift + click to select a range
663da68
Add information about when tests should be added
Trenly Jul 28, 2026
3d43afa
Strengthen wording to request both test types or justification
Trenly Jul 28, 2026
745b5b1
Add coding standards document
Trenly Jul 29, 2026
3b9001b
Add policy regarding AI Assistance
Trenly Jul 29, 2026
66c37ed
Update PR template: add evidence prompt, tests, and AI disclosure
Trenly Jul 29, 2026
9fdbc84
Add Allman, Downcasting, NRVO to spell check allowlist
Trenly Jul 29, 2026
c169a4b
Merge remote tracking branch upstream/master from microsoft/winget-cli
Trenly Sep 22, 2026
71feb9b
Remove note about copyright headers
Trenly Sep 22, 2026
f359933
Fix Typo
Trenly Sep 22, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
8 changes: 7 additions & 1 deletion .github/PULL_REQUEST_TEMPLATE.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,7 @@
<!-- Link related issues, PRs, or docs. Use "Resolves #1234" to auto-close. -->

## 🔍 Validation
<!-- How did you test? List manual steps or note automated test coverage. -->
<!-- How did you test? List manual steps or note automated test coverage. If applicable, include a screenshot or short recording demonstrating the change working. -->

## ✅ Checklist
<!-- Place an "x" between the brackets to check an item. e.g: [x] -->
Expand All @@ -14,8 +14,14 @@
- [ ] Linked to an issue
- [ ] Updated [Release Notes](../doc/ReleaseNotes.md) (if applicable)
- [ ] Updated documentation (if applicable)
- [ ] Added or updated tests (or noted why not applicable)
- [ ] Updated [Copilot instructions](.github/copilot-instructions.md) (if build, architecture, or conventions changed)

## 🤖 AI Assistance
<!-- Check one -->
- [ ] AI assistance was used and has been disclosed in this PR
- [ ] No AI assistance was used

## 📋 Issue Type
<!-- Select the type that best describes this PR -->
- [ ] Bug fix
Expand Down
3 changes: 3 additions & 0 deletions .github/actions/spelling/expect.txt
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -13,6 +13,7 @@ admx
AFAIK
aicli
AICLIC
Allman
allusers
alreadyinstalled
AMap
Expand Down Expand Up @@ -148,6 +149,7 @@ DMC
dnld
Dns
Dobbeleer
Downcasting
DONOT
dsc
dupenv
Expand Down Expand Up @@ -400,6 +402,7 @@ NOUPDATE
nowarn
npmjs
nsis
NRVO
NTFS
objbase
objidl
Expand Down
Loading
Loading