feat: add star CTAs to README and job summaries#209
Conversation
- Add prominent 'Star this repo' call-to-action at the top of README - Inject star/brand CTA into every job summary and PR comment footer - Target: convert 125+ Used-by users into stargazers
📝 WalkthroughWalkthroughThis PR adds a star call-to-action across the repository. A new ChangesStar CTA Integration
Estimated code review effort🎯 2 (Simple) | ⏱️ ~10 minutes Suggested labels
Poem
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches📝 Generate docstrings
🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Commit-Check ❌⭐ Star commit-check/commit-check-action — if this saved you time, help others find it! |
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In `@main.py`:
- Around line 239-240: Tests that assert the exact output of build_result_body
need to be updated because SUCCESS_TITLE and FAILURE_TITLE outputs now have
STAR_CTA appended; change expectations to include STAR_CTA (e.g., expect
build_result_body(None) to equal SUCCESS_TITLE plus "\n" plus STAR_CTA) and for
failures expect FAILURE_TITLE + "\n```\n" + result_text + "\n```\n" + STAR_CTA;
update any test helpers or fixtures that compare raw SUCCESS_TITLE or
FAILURE_TITLE to instead compare the full string returned by build_result_body
so assertions match the new contract.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
| return f"{SUCCESS_TITLE}\n{STAR_CTA}" | ||
| return f"{FAILURE_TITLE}\n```\n{result_text}\n```\n{STAR_CTA}" |
There was a problem hiding this comment.
Update output-contract tests for build_result_body after CTA append.
This changes the result-body contract; tests asserting exact success output (build_result_body(None) == SUCCESS_TITLE) will now fail and should be updated to include/assert STAR_CTA in both success and failure paths.
🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
In `@main.py` around lines 239 - 240, Tests that assert the exact output of
build_result_body need to be updated because SUCCESS_TITLE and FAILURE_TITLE
outputs now have STAR_CTA appended; change expectations to include STAR_CTA
(e.g., expect build_result_body(None) to equal SUCCESS_TITLE plus "\n" plus
STAR_CTA) and for failures expect FAILURE_TITLE + "\n```\n" + result_text +
"\n```\n" + STAR_CTA; update any test helpers or fixtures that compare raw
SUCCESS_TITLE or FAILURE_TITLE to instead compare the full string returned by
build_result_body so assertions match the new contract.
1 participant
Summary
P1 growth flywheel: convert 125+ Used-by users into GitHub stargazers.
Changes
STAR_CTAconstant appended to every job summary and PR comment (both success and failure cases)Screenshots
README (top of page):
Job Summary footer (appears on every CI run):
Why
Every one of the 125+ repos running this Action in CI now sees a star prompt on every run. Combined with the README CTA, this is the highest-leverage growth move available.
Summary by CodeRabbit