Skip to content

feat: redesign the comment footer with website, star and sponsor links - #473

Merged
2bndy5 merged 3 commits into
mainfrom
claude/comment-footer-redesign-a6ulwn
Oct 7, 2026
Merged

2bndy5 merged 3 commits into
mainfrom
claude/comment-footer-redesign-a6ulwn

Conversation

@shenxianpeng

@shenxianpeng shenxianpeng commented Oct 7, 2026 •

Copy link
Copy Markdown
Member

Requested by Xianpeng · project thread

Before: every thread comment, PR review summary and step summary ended with a plain sentence, "Have any feedback or feature suggestions? Share it here.", which only pointed at the issue tracker.

After: they end with a small, muted footer under a rule that links the website, a Star link for cpp-linter-action, the Open Collective sponsor page and the feedback tracker.

Before / after (rendered by GitHub right here)

Before

…report content…

Have any feedback or feature suggestions? Share it here.

After

…report content…


✨ Powered by cpp-linter -- ⭐ Star • ❤️ Sponsor • 💬 Feedback

Full-comment screenshots of both versions are posted in the project thread linked above.

How

Only the USER_OUTREACH constant changes (306 characters, up from 117; the comment-length budget already subtracts it). The footer is pure ASCII (emoji shortcodes and HTML entities) so the existing length-budget math and the escaped-string assertions in the tests keep working unchanged. The sponsor link uses the Open Collective account already listed in cpp-linter/.github/.github/FUNDING.yml. The same text is used in cpp-linter/cpp-linter (cpp_linter/rest_api/__init__.py) so both implementations stay identical; review feedback on cpp-linter/cpp-linter#228 (•, --, emojis inside link text) is applied here too.

Tested locally: cargo fmt --check clean; cargo test -p cpp-linter --all-features for the rest_client unit tests and the comments and reviews integration tests all pass (run with --test-threads=1, which matches nextest isolation).

🤖 Generated with Claude Code

https://claude.ai/code/session_01LUbn11bvXW3Cww9CETPXC4

Summary by CodeRabbit

  • Updates
    • Updated the feedback footer to include cpp-linter attribution and links to star the project, provide sponsorship, and share feedback. These options are now presented together in the footer.

@shenxianpeng shenxianpeng self-assigned this Oct 7, 2026
@codecov

codecov Bot commented Oct 7, 2026 •

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 92.74%. Comparing base (268ddb0) to head (a79d154).

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #473   +/-   ##
=======================================
  Coverage   92.74%   92.74%           
=======================================
  Files          23       23           
  Lines        3859     3859           
=======================================
  Hits         3579     3579           
  Misses        280      280           

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@shenxianpeng
shenxianpeng marked this pull request as ready for review October 7, 2026 07:29
@shenxianpeng
shenxianpeng requested a review from 2bndy5 October 7, 2026 07:29
@coderabbitai

coderabbitai Bot commented Oct 7, 2026 •

Copy link
Copy Markdown
Contributor

Review in Change Stack →

Important

Review skipped

Review was skipped as selected files did not have any reviewable changes.

⚙️ Run configuration
  • Configuration used: Repository UI (base), Organization UI (inherited)
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: ca3680a9-bac3-4c9a-bac9-dacbd614bde3
📥 Commits

Reviewing files that changed from the base of the PR and between 8512c88 and a79d154.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration
  • Configuration used: Repository UI (base), Organization UI (inherited)
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: 2b1b6e22-3a74-49f3-8f4b-20b281a734f5
📥 Commits

Reviewing files that changed from the base of the PR and between 268ddb0 and 8512c88.

📒 Files selected for processing (1)
  • cpp-linter/src/rest_client.rs

Included review availability: This review used your included allowance. Your plan provides up to 1 included review per hour; 0 remain after this review.


📝 Walkthrough

Walkthrough

USER_OUTREACH replaces its feedback prompt and link with a footer that includes cpp-linter attribution, a star link, a sponsor link, and the feedback link.

Changes

Outreach Footer

Layer / File(s) Summary
Update outreach footer
cpp-linter/src/rest_client.rs
USER_OUTREACH now includes cpp-linter attribution and links to star, sponsor, and provide feedback.

Priority: ⬇️ Low

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

Merge Risk: ⚪ Minimal · up to 8512c

Generated comments gain cpp-linter and community links. The supplied evidence indicates no specific user-facing or operational regression that needs resolution before merging.

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly describes the main change: redesigning the comment footer with website, star, and sponsor links.
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 1…
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.
  • Autopilot · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

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.

❤️ Share

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

@shenxianpeng shenxianpeng added the enhancement New feature or request label Oct 7, 2026
shenxianpeng and others added 3 commits October 7, 2026 08:41
Replace the plain "Share it here" feedback line with a compact, muted
footer that links the website, the cpp-linter-action repo, Open
Collective and the issue tracker.

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01LUbn11bvXW3Cww9CETPXC4
Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01LUbn11bvXW3Cww9CETPXC4
Use &bull; and -- instead of &middot;/&mdash; and move the emojis into
the link text; drop the now-unneeded cspell entry.

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01LUbn11bvXW3Cww9CETPXC4
@shenxianpeng
shenxianpeng force-pushed the claude/comment-footer-redesign-a6ulwn branch from 8512c88 to a79d154 Compare October 7, 2026 08:41
@2bndy5
2bndy5 merged commit a268305 into main Oct 7, 2026
73 checks passed
@2bndy5
2bndy5 deleted the claude/comment-footer-redesign-a6ulwn branch October 7, 2026 15:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants