From 2d59ce0543811c738e0ef17d32bc9c32ad49afab Mon Sep 17 00:00:00 2001 From: Rituraj <56511165+imxade@users.noreply.github.com> Date: Tue, 24 Feb 2026 21:09:50 +0530 Subject: [PATCH 1/2] Update CONTRIBUTING.md with review and issue guidelines Added guidelines for reviewing PRs and issue assignment. --- CONTRIBUTING.md | 13 ++++++++++++- 1 file changed, 12 insertions(+), 1 deletion(-) diff --git a/CONTRIBUTING.md b/CONTRIBUTING.md index 32a0fbf..33d39a0 100644 --- a/CONTRIBUTING.md +++ b/CONTRIBUTING.md @@ -172,6 +172,7 @@ git push origin feature/your-feature-name - [ ] You've updated relevant documentation - [ ] Your commits are clean and well-organized - [ ] You've rebased with the latest upstream changes +- [ ] You've thought from the reviewer's perspective and made your PR easy to review ### Submitting a Pull Request @@ -213,6 +214,12 @@ Steps to test the changes - Respond to review comments promptly - Make requested changes in new commits - Be patient - maintainers will review when available +- Use `[WIP]` in your PR title for incomplete PRs. Don't use this as a way to gatekeep; focus on one change until it gets merged. + +### Reviewing PRs + +- Instead of opening duplicate PRs, help review and improve existing ones. +- When reviewing, assess whether the change is actually necessary before diving into implementation details and functionality testing. ## 📝 Code Style Guidelines @@ -225,6 +232,7 @@ TODO: Add project-specific code style guidelines - Add comments for complex logic - Remove console.logs before committing - Avoid code duplication +- Avoid unnecessary complexity and minor over-optimization ### JavaScript/TypeScript - Use ES6+ syntax @@ -521,9 +529,12 @@ If you encounter issues not covered here: ## 🎯 Issue Assignment - One contributor per issue (unless specified otherwise) + +- If there are no active PRs for an issue for 2+ days, mention your intent under the issue and begin +- Check for existing PRs before starting to avoid duplication, as there might PRs that didn't mention the related issue Thank you for contributing to TODO! Your efforts help make this project better for everyone. 🚀 From c9a550da54a08f0eb684a78bb1cd71dcce9b2a02 Mon Sep 17 00:00:00 2001 From: Rituraj <56511165+imxade@users.noreply.github.com> Date: Wed, 25 Feb 2026 12:22:20 +0530 Subject: [PATCH 2/2] Update CONTRIBUTING.md with clearer guidelines Clarify contribution guidelines regarding issue assignment and PRs. --- CONTRIBUTING.md | 1 + 1 file changed, 1 insertion(+) diff --git a/CONTRIBUTING.md b/CONTRIBUTING.md index 33d39a0..8b5300b 100644 --- a/CONTRIBUTING.md +++ b/CONTRIBUTING.md @@ -534,6 +534,7 @@ If you encounter issues not covered here: - Issues will be reassigned if inactive for extended periods --> - If there are no active PRs for an issue for 2+ days, mention your intent under the issue and begin +- Avoid working on issues which are assigned to someone, even if they are inactive - Check for existing PRs before starting to avoid duplication, as there might PRs that didn't mention the related issue