-
Notifications
You must be signed in to change notification settings - Fork 0
Update README.md #104
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Update README.md #104
Changes from all commits
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change | ||||||||||||||||||||||||||||||||||||||||||||
|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
|
|
@@ -9,6 +9,57 @@ | |||||||||||||||||||||||||||||||||||||||||||||
|  | ||||||||||||||||||||||||||||||||||||||||||||||
|  | ||||||||||||||||||||||||||||||||||||||||||||||
|  | ||||||||||||||||||||||||||||||||||||||||||||||
|  | ||||||||||||||||||||||||||||||||||||||||||||||
|  | ||||||||||||||||||||||||||||||||||||||||||||||
|  | ||||||||||||||||||||||||||||||||||||||||||||||
|  | ||||||||||||||||||||||||||||||||||||||||||||||
|  | ||||||||||||||||||||||||||||||||||||||||||||||
|  | ||||||||||||||||||||||||||||||||||||||||||||||
|
Comment on lines
+16
to
+17
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. suggestion: Both contributors badges use the same alt text, even though one includes anonymous contributors. Since one badge includes anonymous contributors, please differentiate the alt text (for example,
Suggested change
|
||||||||||||||||||||||||||||||||||||||||||||||
|  | ||||||||||||||||||||||||||||||||||||||||||||||
|  | ||||||||||||||||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||||||||||||||||
| CodeRabbit | ||||||||||||||||||||||||||||||||||||||||||||||
|
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Title: Badges reference other repositories instead of this Tools project Category: Problem: Several newly added badges in Evidence:
The rest of the README (lines 1–20, 65–93) consistently uses Impact: The README displays quality, CI, and coverage status for other projects, not for this codebase. This misleads contributors and maintainers about the actual health and checks of the 💡 Recommendation:
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Title: Badges reference other repositories instead of this Tools project Category: Problem: Several newly added badges in Evidence:
The rest of the README (lines 1–19, 65–93, 95–103) consistently uses Impact:
💡 Recommendation:
This keeps the README’s status badges accurate for the |
||||||||||||||||||||||||||||||||||||||||||||||
|  | ||||||||||||||||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||||||||||||||||
| CodeFactor | ||||||||||||||||||||||||||||||||||||||||||||||
| [](https://www.codefactor.io/repository/github/lcsogthb/games) | ||||||||||||||||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||||||||||||||||
| Codecov | ||||||||||||||||||||||||||||||||||||||||||||||
| [](https://codecov.io/github/LCSOGthb/Games) | ||||||||||||||||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||||||||||||||||
| CircleCI | ||||||||||||||||||||||||||||||||||||||||||||||
|
Comment on lines
+21
to
+30
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. issue (bug_risk): Many of the new badges link to other repositories/projects (Games/Clown) instead of this Tools repo. These badges still reference |
||||||||||||||||||||||||||||||||||||||||||||||
| [](https://dl.circleci.com/status-badge/redirect/gh/LCSOGthb/Games/tree/main) | ||||||||||||||||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||||||||||||||||
| DeepSource | ||||||||||||||||||||||||||||||||||||||||||||||
| [](https://app.deepsource.com/gh/LCSOGthb/Games/) | ||||||||||||||||||||||||||||||||||||||||||||||
| [](https://app.deepsource.com/gh/LCSOGthb/Games/) | ||||||||||||||||||||||||||||||||||||||||||||||
| [](https://app.deepsource.com/gh/LCSOGthb/Games/) | ||||||||||||||||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||||||||||||||||
| OSSF | ||||||||||||||||||||||||||||||||||||||||||||||
| [](https://scorecard.dev/viewer/?uri=github.com/LCSOGthb/Games) | ||||||||||||||||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||||||||||||||||
| Qlty | ||||||||||||||||||||||||||||||||||||||||||||||
| [](https://qlty.sh/gh/LCSOGthb/projects/Games) | ||||||||||||||||||||||||||||||||||||||||||||||
| [](https://qlty.sh/gh/LCSOGthb/projects/Games) | ||||||||||||||||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||||||||||||||||
| Scrutinizer | ||||||||||||||||||||||||||||||||||||||||||||||
| [](https://scrutinizer-ci.com/g/LCSOGthb/Games/?branch=main) | ||||||||||||||||||||||||||||||||||||||||||||||
| [](https://scrutinizer-ci.com/g/LCSOGthb/Games/?branch=main) | ||||||||||||||||||||||||||||||||||||||||||||||
| [](https://scrutinizer-ci.com/g/LCSOGthb/Games/build-status/main) | ||||||||||||||||||||||||||||||||||||||||||||||
| [](https://scrutinizer-ci.com/code-intelligence) | ||||||||||||||||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||||||||||||||||
|
Comment on lines
+22
to
+50
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Badge URLs point to a different repository ( These badges currently report metrics for another repo, so README status is misleading for this project. 🔧 Proposed fix-
+
-[](https://www.codefactor.io/repository/github/lcsogthb/games)
+[](https://www.codefactor.io/repository/github/lcsogthb/tools)
-[](https://codecov.io/github/LCSOGthb/Games)
+[](https://codecov.io/github/LCSOGthb/Tools)
-[](https://dl.circleci.com/status-badge/redirect/gh/LCSOGthb/Games/tree/main)
+[](https://dl.circleci.com/status-badge/redirect/gh/LCSOGthb/Tools/tree/main)
-[](https://app.deepsource.com/gh/LCSOGthb/Games/)
+[](https://app.deepsource.com/gh/LCSOGthb/Tools/)🤖 Prompt for AI Agents |
||||||||||||||||||||||||||||||||||||||||||||||
| SonarQube | ||||||||||||||||||||||||||||||||||||||||||||||
| [](https://sonarcloud.io/summary/new_code?id=LCSOGthb_Clown) | ||||||||||||||||||||||||||||||||||||||||||||||
| [](https://sonarcloud.io/summary/new_code?id=LCSOGthb_Clown) | ||||||||||||||||||||||||||||||||||||||||||||||
| [](https://sonarcloud.io/summary/new_code?id=LCSOGthb_Clown) | ||||||||||||||||||||||||||||||||||||||||||||||
| [](https://sonarcloud.io/summary/new_code?id=LCSOGthb_Clown) | ||||||||||||||||||||||||||||||||||||||||||||||
| [](https://sonarcloud.io/summary/new_code?id=LCSOGthb_Clown) | ||||||||||||||||||||||||||||||||||||||||||||||
| [](https://sonarcloud.io/summary/new_code?id=LCSOGthb_Clown) | ||||||||||||||||||||||||||||||||||||||||||||||
| [](https://sonarcloud.io/summary/new_code?id=LCSOGthb_Clown) | ||||||||||||||||||||||||||||||||||||||||||||||
| [](https://sonarcloud.io/summary/new_code?id=LCSOGthb_Clown) | ||||||||||||||||||||||||||||||||||||||||||||||
| [](https://sonarcloud.io/summary/new_code?id=LCSOGthb_Clown) | ||||||||||||||||||||||||||||||||||||||||||||||
| [](https://sonarcloud.io/summary/new_code?id=LCSOGthb_Clown) | ||||||||||||||||||||||||||||||||||||||||||||||
| [](https://sonarcloud.io/summary/new_code?id=LCSOGthb_Clown) | ||||||||||||||||||||||||||||||||||||||||||||||
|
Comment on lines
+22
to
+62
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. 🚩 Badges reference wrong repositories (Games and Clown instead of Tools) The newly added badges at lines 22-62 reference Was this helpful? React with 👍 or 👎 to provide feedback.
Comment on lines
+52
to
+62
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. SonarQube badge project key is inconsistent with configured analysis project. README uses 🔧 Proposed fix-[](https://sonarcloud.io/summary/new_code?id=LCSOGthb_Clown)
+[](https://sonarcloud.io/summary/new_code?id=LCSOGthb_Tools)Apply the same 📝 Committable suggestion
Suggested change
🤖 Prompt for AI Agents |
||||||||||||||||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||||||||||||||||
| CodeFactor: | ||||||||||||||||||||||||||||||||||||||||||||||
| [](https://www.codefactor.io/repository/github/lcsogthb/tools) | ||||||||||||||||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||||||||||||||||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
suggestion: All four commit-activity badges share identical alt text, which may be confusing.
Consider updating each badge’s alt text to reflect its time period, for example:
GitHub weekly commit activity,GitHub monthly commit activity,GitHub yearly commit activity, andGitHub total commit activity, so screen readers and non-visual contexts can distinguish them.