Skip to content

Update contribution guidelines and templates#303

Merged
mor39a merged 4 commits into
mainfrom
update-contributing
May 5, 2026
Merged

Update contribution guidelines and templates#303
mor39a merged 4 commits into
mainfrom
update-contributing

Conversation

@Kyome22
Copy link
Copy Markdown
Collaborator

@Kyome22 Kyome22 commented May 5, 2026

Context of Contribution

  • Bug Fix
  • Refactoring
  • New Feature
  • Others

Summary of the Proposal

Updates the contribution guidelines and the issue / pull request templates to make expectations around contributing clearer.

  • Added a Responding to Review Comments subsection to CONTRIBUTING.md:
    • Do not click Resolve conversation unless you are the reviewer, since only the reviewer can judge whether the issue is actually resolved.
    • Address each review comment individually rather than bundling responses, since larger diffs make it harder for the reviewer to evaluate the change.
  • Added an I have read the CONTRIBUTING.md and agree to follow it required checklist item to:
    • .github/pull_request_template.md
    • .github/ISSUE_TEMPLATE/bug_report.yml
    • .github/ISSUE_TEMPLATE/feature_request.yml

Reason for the new feature

N/A (documentation / template change).

Checklist

  • I have read the CONTRIBUTING.md and agree to follow it.
  • This PR does not contain commits of multiple contexts.
  • Code follows proper indentation and naming conventions.
  • Implemented using only APIs that can be submitted to the Microsoft Store.
  • Works correctly in both dark theme and light theme.
  • Works correctly on any device.

🤖 Generated with Claude Code

Kyome22 and others added 2 commits May 5, 2026 17:23
Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
Comment thread CONTRIBUTING.md
Comment thread CONTRIBUTING.md
Kyome22 and others added 2 commits May 5, 2026 23:57
Co-authored-by: Santiago Mora <89531894+mor39a@users.noreply.github.com>
Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
Copy link
Copy Markdown
Collaborator

@mor39a mor39a left a comment

Choose a reason for hiding this comment

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

LGTM

@mor39a mor39a merged commit 81bf38f into main May 5, 2026
@mor39a mor39a deleted the update-contributing branch May 5, 2026 15:43
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