Skip to content

docs: link the App's install page, not its Marketplace listing - #292

Merged
shenxianpeng merged 1 commit into
mainfrom
chore/link-app-install-page
Sep 25, 2026
Merged

shenxianpeng merged 1 commit into
mainfrom
chore/link-app-install-page

Conversation

@shenxianpeng

@shenxianpeng shenxianpeng commented Sep 25, 2026 •

Copy link
Copy Markdown
Member

Links to the Commit Check App now go to https://github.com/apps/commit-check instead of https://github.com/marketplace/commit-check.

The Marketplace listing has been under review since 2026-09-02. Until it is approved, the Marketplace URL is a 404 for anyone not signed in. The App page is public and installs the same App, so these links work for every reader today. They can move back to the listing once it is live.

Summary by CodeRabbit

  • Documentation
    • Updated Commit Check links to point to its dedicated GitHub App page in the README and fork-PR guidance.

github.com/marketplace/commit-check is a 404 for anyone who is not signed
in while the listing is still under review, which it has been since
2026-09-02. github.com/apps/commit-check is public and installs the same
App, so the links point there until the listing is live.
@shenxianpeng
shenxianpeng requested a review from a team as a code owner September 25, 2026 20:32
@coderabbitai

coderabbitai Bot commented Sep 25, 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: defaults

Review profile: CHILL

Plan: Advanced

Run ID: 7297d5c5-1907-481a-b8cb-98f0be1a44db

📥 Commits

Reviewing files that changed from the base of the PR and between 7004ced and 7391cd0.

📒 Files selected for processing (2)
  • README.md
  • docs/fork-pr-comments.md

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


📝 Walkthrough

Walkthrough

Three links in the README and fork-PR documentation now point to the Commit Check GitHub App page instead of its Marketplace listing.

Changes

GitHub App links

Layer / File(s) Summary
Update Commit Check app links
README.md, docs/fork-pr-comments.md
Three links now point to github.com/apps/commit-check instead of the Marketplace listing.

Priority: ➖ Normal

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

Change: Other

Merge Risk: ⚪ Minimal · up to 7391c

The documentation links now point to the App page as intended. No material merge risk is evident.

Architecture Summary

Architecture risk: 🔵 Low · up to 7391c

The change affects 2 systems.

Changed systems: docs, README.md

Architecture concerns
No architecture-level concerns identified.

Review details

Systems and components

  • observed — docs (service) was modified; 1 changed file maps to changed impact.
  • observed — README.md (service) was modified; 1 changed file maps to changed impact.

Before / after behavior

  • observed — Modified behavior in README.md: The GitHub App link in the comparison table now points to github.com/apps/commit-check instead of the Marketplace page.
  • observed — Modified behavior in README.md: The GitHub App link in the fork-PR guidance now points to github.com/apps/commit-check instead of the Marketplace page.
  • observed — Modified behavior in docs/fork-pr-comments.md: Changed the Commit Check GitHub App link from the Marketplace listing to github.com/apps/commit-check.
🚥 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 and concisely describes the main documentation change: replacing the Marketplace listing link with the App install page.
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)
  • Commit to this branch
  • Create a new PR

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.

@github-actions

Copy link
Copy Markdown
Contributor

Commit Check

✅ All 5 checks passed

Show all 5 checks
Commit message
  ✔ PR title (docs: link the App's install page, not its Marketplace li...)
  ✔ Commit 1/1 (7391cd0) (docs: link the App's install page, not its Marketplace li...)
Branch
  ✔ Branch (chore/link-app-install-page)
Author
  ✔ Author name (Xianpeng Shen)
  ✔ Author email (xianpeng.shen@gmail.com)

commit-check 2.18.1 · Rules reference

@codecov

codecov Bot commented Sep 25, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 95.11%. Comparing base (7004ced) to head (7391cd0).

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #292   +/-   ##
=======================================
  Coverage   95.11%   95.11%           
=======================================
  Files           1        1           
  Lines         614      614           
=======================================
  Hits          584      584           
  Misses         30       30           
Flag Coverage Δ
unittests 95.11% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ 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.

@shenxianpeng
shenxianpeng merged commit d9f3b9d into main Sep 25, 2026
22 checks passed
@shenxianpeng
shenxianpeng deleted the chore/link-app-install-page branch September 25, 2026 20:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant