Skip to content

Conversation

@castrojo
Copy link
Contributor

This adds the health check, contributor count, and contributor count (yearly) to the badges section of the readme.

LFX Health Score
LFX Contributors
LFX Active Contributors

This adds the health check, contributor count, and contributor count (yearly) to the badges section of the readme.
@github-actions github-actions bot added documentation control/skip-ci Do not run expensive CI on this job labels Jul 31, 2025
Copy link
Contributor

@gemini-code-assist gemini-code-assist bot left a comment

Choose a reason for hiding this comment

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

Code Review

This pull request adds LFX Insights badges to the README.md, which is a great way to showcase project health and activity. My review includes a suggestion to enhance the accessibility of the new badges by making their alt text more descriptive.

Comment on lines +64 to +66
[![LFX Health Score](https://insights.linuxfoundation.org/api/badge/health-score?project=bootc)](https://insights.linuxfoundation.org/project/bootc)
[![LFX Contributors](https://insights.linuxfoundation.org/api/badge/contributors?project=bootc)](https://insights.linuxfoundation.org/project/bootc)
[![LFX Active Contributors](https://insights.linuxfoundation.org/api/badge/active-contributors?project=bootc)](https://insights.linuxfoundation.org/project/bootc)
Copy link
Contributor

Choose a reason for hiding this comment

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

medium

To improve accessibility and provide more context for screen readers, it's a good practice to include the project name in the alt text for the badges. This makes it clearer what each badge refers to, especially when the content is viewed out of its original context.

Suggested change
[![LFX Health Score](https://insights.linuxfoundation.org/api/badge/health-score?project=bootc)](https://insights.linuxfoundation.org/project/bootc)
[![LFX Contributors](https://insights.linuxfoundation.org/api/badge/contributors?project=bootc)](https://insights.linuxfoundation.org/project/bootc)
[![LFX Active Contributors](https://insights.linuxfoundation.org/api/badge/active-contributors?project=bootc)](https://insights.linuxfoundation.org/project/bootc)
[![bootc LFX Health Score](https://insights.linuxfoundation.org/api/badge/health-score?project=bootc)](https://insights.linuxfoundation.org/project/bootc)
[![bootc LFX Contributors](https://insights.linuxfoundation.org/api/badge/contributors?project=bootc)](https://insights.linuxfoundation.org/project/bootc)
[![bootc LFX Active Contributors](https://insights.linuxfoundation.org/api/badge/active-contributors?project=bootc)](https://insights.linuxfoundation.org/project/bootc)

Copy link
Collaborator

Choose a reason for hiding this comment

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

Fascinating...I wonder if that bit is in the system prompt for the review app or whether it really just inferred that.

Copy link
Collaborator

Choose a reason for hiding this comment

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

Bigger picture it'd be cool to have a template we pull in/sync from somewhere automatically that does this stuff.

@cgwalters cgwalters merged commit 6c3bba5 into bootc-dev:main Jul 31, 2025
27 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

control/skip-ci Do not run expensive CI on this job

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants