From c3c698fd94d403950005e34ef158bd7c2b39ce9c Mon Sep 17 00:00:00 2001 From: tuanwannafly Date: Sun, 4 Jan 2026 22:41:17 +0700 Subject: [PATCH 1/2] Add TuanWanna to CLA signers --- .github/cla-signers.json | 10 ++++++++++ 1 file changed, 10 insertions(+) diff --git a/.github/cla-signers.json b/.github/cla-signers.json index aa95fef8..0f8e0877 100644 --- a/.github/cla-signers.json +++ b/.github/cla-signers.json @@ -149,7 +149,17 @@ ], "signed_date": "2026-01-03", "cla_version": "1.0" + }, + { + "name": "Tuan Wanna", + "github_username": "tuanwannafly", + "emails": [ + "tunawillgetit1030@gmail.com" + ], + "signed_date": "2026-01-03", + "cla_version": "1.0" } + ], "corporations": { "AI Venture Holdings LLC": { From fef216036a44e9933facd4594eb8dc09e1adbb53 Mon Sep 17 00:00:00 2001 From: tuanwannafly Date: Sun, 4 Jan 2026 23:19:18 +0700 Subject: [PATCH 2/2] add Tuna to cla --- .github/cla-signers.json | 10 ++++++++++ .github/pull_request_template.md | 7 +++++++ CONTRIBUTING.md | 27 ++++++++++++++++++++++++++- 3 files changed, 43 insertions(+), 1 deletion(-) diff --git a/.github/cla-signers.json b/.github/cla-signers.json index 0f8e0877..f9aaf5e6 100644 --- a/.github/cla-signers.json +++ b/.github/cla-signers.json @@ -150,6 +150,15 @@ "signed_date": "2026-01-03", "cla_version": "1.0" }, + { + "name": "Rakesh Jha", + "github_username": "rakesh0x", + "emails": [ + "rakeshjhanda9958@gmail.com" + ], + "signed_date": "2026-01-03", + "cla_version": "1.0" + }, { "name": "Tuan Wanna", "github_username": "tuanwannafly", @@ -160,6 +169,7 @@ "cla_version": "1.0" } + ], "corporations": { "AI Venture Holdings LLC": { diff --git a/.github/pull_request_template.md b/.github/pull_request_template.md index 50feebfd..a9327184 100644 --- a/.github/pull_request_template.md +++ b/.github/pull_request_template.md @@ -4,6 +4,13 @@ Closes # ## Summary +## AI Disclosure + +- [ ] No AI used +- [ ] AI/IDE/Agents used (please describe below) + + + ## Checklist - [ ] Tests pass (`pytest tests/`) - [ ] MVP label added if closing MVP issue diff --git a/CONTRIBUTING.md b/CONTRIBUTING.md index 76aa0e33..a2edaebf 100644 --- a/CONTRIBUTING.md +++ b/CONTRIBUTING.md @@ -24,6 +24,7 @@ We are committed to providing a welcoming and inclusive environment. Please: - Be respectful and considerate - Use welcoming and inclusive language +- Write in clear, professional English in all communications (issues, PRs, comments, and documentation) - Accept constructive criticism gracefully - Focus on what's best for the community - Show empathy towards other community members @@ -179,7 +180,8 @@ pip install pytest pytest-cov pytest-mock black pylint mypy bandit - `bounty` - Has cash reward - `priority:high` - Important issues 3. Comment "I'd like to work on this" to claim an issue -4. Wait for assignment before starting (prevents duplicate work) +4. **Wait for assignment before starting work** (prevents duplicate effort) +5. Once assigned, please raise your PR within approximately one week. If there's no progress or update within this timeframe, we may unassign the issue to give others a chance to contribute. If you need more time, just let us know in the issue comments. --- @@ -216,6 +218,13 @@ Closes #123 - [ ] Breaking change - [ ] Documentation update +## AI Disclosure +Please indicate if you used AI tools during development: +- [ ] No AI used +- [ ] AI/IDE/Agents used (please describe below) + +If AI was used, briefly explain which tools and how they assisted (e.g., "Used GitHub Copilot for autocomplete suggestions" or "Used ChatGPT to help debug a regex pattern"). + ## Testing How did you test these changes? @@ -225,6 +234,22 @@ How did you test these changes? - [ ] Documentation updated ``` +### Git Workflow + +**Please do not force push to your branch.** Always use merge commits to incorporate upstream changes: + +```bash +# Keep your branch up to date with main +git fetch upstream +git merge upstream/main + +# Do NOT use: +# git push --force +# git rebase upstream/main (followed by force push) +``` + +Merge commits preserve history and make it easier for reviewers to track changes. Force pushing can cause confusion and may overwrite collaborative work. + ### Review Process 1. **CLA verification** - Bot checks for signature