Skip to content

docs: fix dead ktlint badge link in README - #648

Closed
1t1sCooL wants to merge 1 commit into
iamgio:mainfrom
1t1sCooL:fix-ktlint-badge-link
Closed

docs: fix dead ktlint badge link in README#648
1t1sCooL wants to merge 1 commit into
iamgio:mainfrom
1t1sCooL:fix-ktlint-badge-link

Conversation

@1t1sCooL

@1t1sCooL 1t1sCooL commented Aug 27, 2026

Copy link
Copy Markdown

The ktlint badge link is dead — ktlint moved from the pinterest org to the ktlint org (the old repo redirects), so https://pinterest.github.io/ktlint returns 404. Replaced with the project's current docs URL declared by the repo: https://ktlint.github.io/ktlint/ (HTTP 200; note the bare ktlint.github.io root still meta-refreshes to the dead URL, so the /ktlint/ path is the correct target).

Summary by CodeRabbit

  • Documentation
    • Updated the Ktlint badge link to point to the official Ktlint documentation.

@coderabbitai

coderabbitai Bot commented Aug 27, 2026

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro Plus

Run ID: 4f54e448-923d-4928-b19d-2205032b4f43

📥 Commits

Reviewing files that changed from the base of the PR and between 7d4cac3 and 4ad9598.

📒 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 Ktlint badge now links to the official Ktlint documentation. The badge image and label remain unchanged.

Changes

Documentation update

Layer / File(s) Summary
Update Ktlint badge destination
README.md
The Ktlint badge link uses https://ktlint.github.io/ktlint/.

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

Merge Risk: ⚪ Minimal · up to 4ad95

This localized documentation change updates a dead ktlint badge link to the current documentation URL and introduces no actionable merge-blocking risk beyond normal checks.

Suggested reviewers: iamgio

Poem

A rabbit checks the badge with care
The Ktlint link now leads elsewhere
The image stays bright
The label stays right
Clean paths hop through the air

🚥 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 change: fixing the dead Ktlint badge link in README.md.
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.
Full details: Docstring Coverage

Explanation

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 files. (1 skipped: 1 unsupported.)

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

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.

@1t1sCooL

1t1sCooL commented Sep 2, 2026

Copy link
Copy Markdown
Author

Closing: the badge link was already updated on main to https://ktlint.github.io/ktlint/latest/, so this PR is no longer needed. Thanks!

@1t1sCooL 1t1sCooL closed this Sep 2, 2026
@iamgio

iamgio commented Sep 2, 2026

Copy link
Copy Markdown
Owner

Weird, I got an email notification for this PR but I couldn't find it or open it as it would 404. That's why I fixed it myself. Sorry about that

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.

2 participants