Skip to content

Add Linux Foundation Health Score to README#24983

Open
jonathimer wants to merge 1 commit intoTryGhost:mainfrom
jonathimer:patch-1
Open

Add Linux Foundation Health Score to README#24983
jonathimer wants to merge 1 commit intoTryGhost:mainfrom
jonathimer:patch-1

Conversation

@jonathimer
Copy link

Congrats! We have onboarded Ghost to LFX Insights, the Linux Foundation's platform for monitoring the world's most critical open-source projects.

https://insights.linuxfoundation.org/project/tryghost-ghost

Congrats! We have onboarded Ghost to LFX Insights, the Linux Foundation's platform for monitoring the world's most critical open-source projects.

https://insights.linuxfoundation.org/project/tryghost-ghost
@coderabbitai
Copy link
Contributor

coderabbitai bot commented Sep 25, 2025

Walkthrough

Adds a new health score badge block to README.md under the Contributors section. The change inserts a link that wraps an image badge sourced from insights.linuxfoundation.org. No existing content is modified or removed, and no code or configuration files are affected.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Pre-merge checks and finishing touches

✅ Passed checks (3 passed)
Check name Status Explanation
Title Check ✅ Passed The pull request title clearly summarizes the addition of the Linux Foundation health score badge to the README, directly reflecting the main change without extraneous detail.
Description Check ✅ Passed The pull request description concisely notes the onboarding of Ghost to LFX Insights and provides the relevant link, directly relating to the change introduced in the README.
Docstring Coverage ✅ Passed No functions found in the changes. Docstring coverage check skipped.
✨ Finishing touches
🧪 Generate unit tests
  • Create PR with unit tests
  • Post copyable unit tests in a comment

Tip

👮 Agentic pre-merge checks are now available in preview!

Pro plan users can now enable pre-merge checks in their settings to enforce checklists before merging PRs.

  • Built-in checks – Quickly apply ready-made checks to enforce title conventions, require pull request descriptions that follow templates, validate linked issues for compliance, and more.
  • Custom agentic checks – Define your own rules using CodeRabbit’s advanced agentic capabilities to enforce organization-specific policies and workflows. For example, you can instruct CodeRabbit’s agent to verify that API documentation is updated whenever API schema files are modified in a PR. Note: Upto 5 custom checks are currently allowed during the preview period. Pricing for this feature will be announced in a few weeks.

Please see the documentation for more information.

Example:

reviews:
  pre_merge_checks:
    custom_checks:
      - name: "Undocumented Breaking Changes"
        mode: "warning"
        instructions: |
          Pass/fail criteria: All breaking changes to public APIs, CLI flags, environment variables, configuration keys, database schemas, or HTTP/GraphQL endpoints must be documented in the "Breaking Change" section of the PR description and in CHANGELOG.md. Exclude purely internal or private changes (e.g., code not exported from package entry points or explicitly marked as internal).

Please share your feedback with us on this Discord post.


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 and usage tips.

@github-actions github-actions bot added the community [triage] Community features and bugs label Sep 25, 2025
Copy link

@cursor cursor bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This PR is being reviewed by Cursor Bugbot

Details

Your team is on the Bugbot Free tier. On this plan, Bugbot will review limited PRs each billing cycle for each member of your team.

To receive Bugbot reviews on all of your PRs, visit the Cursor dashboard to activate Pro and start your 14-day free trial.

<a href="https://github.com/TryGhost/Ghost/contributors/">
<img src="https://img.shields.io/github/contributors/TryGhost/Ghost.svg" alt="Contributors" />
</a>
<a href="[https://github.com/TryGhost/Ghost/contributors/](https://insights.linuxfoundation.org/project/tryghost-ghost)">
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Bug: Markdown Link Syntax in href Causes Navigation Failure

The new <a> tag in README.md includes Markdown link syntax [https://github.com/TryGhost/Ghost/contributors/](https://insights.linuxfoundation.org/project/tryghost-ghost) in its href attribute. This prevents the link from working and navigating to the intended Linux Foundation Insights page.

Fix in Cursor Fix in Web

Copy link
Contributor

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Actionable comments posted: 1

📜 Review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between e0a17c3 and 62f2eb7.

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

Comment on lines +31 to +33
<a href="[https://github.com/TryGhost/Ghost/contributors/](https://insights.linuxfoundation.org/project/tryghost-ghost)">
<img src="https://insights.linuxfoundation.org/api/badge/health-score?project=tryghost-ghost" alt="Health score" />
</a>
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

⚠️ Potential issue | 🟠 Major

Fix the malformed badge link.
The href attribute still contains a markdown-formatted link, so the browser tries to follow [https://github.com/TryGhost/Ghost/contributors/](https://insights.linuxfoundation.org/project/tryghost-ghost) and the badge is effectively broken.

Apply this diff to repair the link:

-    <a href="[https://github.com/TryGhost/Ghost/contributors/](https://insights.linuxfoundation.org/project/tryghost-ghost)">
+    <a href="https://insights.linuxfoundation.org/project/tryghost-ghost">
📝 Committable suggestion

‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.

Suggested change
<a href="[https://github.com/TryGhost/Ghost/contributors/](https://insights.linuxfoundation.org/project/tryghost-ghost)">
<img src="https://insights.linuxfoundation.org/api/badge/health-score?project=tryghost-ghost" alt="Health score" />
</a>
<a href="https://insights.linuxfoundation.org/project/tryghost-ghost">
<img src="https://insights.linuxfoundation.org/api/badge/health-score?project=tryghost-ghost" alt="Health score" />
</a>
🤖 Prompt for AI Agents
In README.md around lines 31 to 33 the badge anchor's href contains a
markdown-style link text instead of the actual URL, breaking the badge link;
replace the href value so it points directly to the intended URL
(https://insights.linuxfoundation.org/project/tryghost-ghost) and ensure the <a>
tag wraps the <img> correctly with no markdown punctuation.

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

Labels

community [triage] Community features and bugs

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant