Skip to content

fix(docs): update contributing section on README.md - #13512

Open
Project516 wants to merge 1 commit into
pingdotgg:mainfrom
Project516:fix/readme-contributing
Open

Project516 wants to merge 1 commit into
pingdotgg:mainfrom
Project516:fix/readme-contributing

Conversation

@Project516

@Project516 Project516 commented Sep 24, 2026 •

Copy link
Copy Markdown

What Changed

Update the contributing section of the README

Why

The old version of the README had a notice that t3code is not accepting prs and then underneath it is where it had the if you still want to contribute. That was removed, so the first contributing section in the README is “if you still want to contribute”.

UI Changes

N/A

Checklist

  • This PR is small and focused
  • I explained what changed and why
  • I included before/after screenshots for any UI changes: N/A
  • I included a video for animation/interaction changes: N/A

Summary by CodeRabbit

  • Documentation
    • Updated the contribution guide heading to “If you want to contribute.... read this first.”

@github-actions github-actions Bot added vouch:unvouched PR author is not yet trusted in the VOUCHED list. size:XS 0-9 changed lines (additions + deletions). labels Sep 24, 2026
@coderabbitai

coderabbitai Bot commented Sep 24, 2026

Copy link
Copy Markdown

Review in Change Stack →

Navigate logical layers of code changes, visualize relationships, and explore their blast radius.

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Repository: pingdotgg/t3code/.coderabbit.yaml

Review profile: CHILL

Plan: Advanced

Run ID: a10cfd20-9d38-49dc-9c18-594116f7c1e2

📥 Commits

Reviewing files that changed from the base of the PR and between 720490a and 8f7a4fb.

📒 Files selected for processing (1)
  • README.md

Included review availability: Your plan provides up to 10 included reviews per hour; 9 remain after this review.


📝 Walkthrough

Walkthrough

The README contribution heading changes from “If you REALLY want to contribute still.... read this first” to “If you want to contribute.... read this first.”

Changes

README contribution heading

Layer / File(s) Summary
Revise contribution heading
README.md
The heading drops “REALLY” and “still.”

Priority: ⬇️ Low

Estimated code review effort: 1 (Trivial) | ~2 minutes

Change: Other

Suggested reviewers: juliusmarminge

Merge Risk: ⚪ Minimal · up to 8f7a4

This updates the contribution heading without changing product behavior. No merge-blocking risk is evident.

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly identifies the documentation change to the README contributing section.
Description check ✅ Passed The description includes all required sections and explains the change, reason, scope, and lack of UI changes.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0…
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create a new PR

Comment @coderabbitai help to get the list of available commands.

@macroscopeapp

macroscopeapp Bot commented Sep 24, 2026

Copy link
Copy Markdown
Contributor

Approvability

Verdict: Approved at 8f7a4fb

Macroscope's review found this PR approvable — This PR only updates the wording of a README contribution heading. It changes no executable code, product behavior, defaults, schemas, or configuration.

You can add or adjust custom eligibility rules. Learn more.

This branch has not been deployed

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

Labels

size:XS 0-9 changed lines (additions + deletions). vouch:unvouched PR author is not yet trusted in the VOUCHED list.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant