-
Notifications
You must be signed in to change notification settings - Fork 3
chore: add contributors #19
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
Conversation
WalkthroughThe README file has been updated to include a new contributor, Changes
Poem
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. 🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
CodeRabbit Configuration File (
|
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.
Actionable comments posted: 0
🧹 Nitpick comments (2)
README.md (2)
66-67: Add Alt Text for Contributor ImagesThe new contributor entry for
yangkeni(and similar entries) currently lacksalttext in the<img>tag. Adding analtattribute will improve accessibility and resolve markdownlint MD045 warnings.
Suggested diff:-|[<img src="https://avatars.githubusercontent.com/u/61226209?v=4" width="100px;"/><br/><sub><b>yangkeni</b></sub>](https://github.com/yangkeni)<br/>| +|[<img src="https://avatars.githubusercontent.com/u/61226209?v=4" alt="yangkeni avatar" width="100px;"/><br/><sub><b>yangkeni</b></sub>](https://github.com/yangkeni)<br/>|🧰 Tools
🪛 markdownlint-cli2 (0.17.2)
66-66: Images should have alternate text (alt text)
null(MD045, no-alt-text)
66-66: Images should have alternate text (alt text)
null(MD045, no-alt-text)
66-66: Images should have alternate text (alt text)
null(MD045, no-alt-text)
66-66: Images should have alternate text (alt text)
null(MD045, no-alt-text)
70-70: Refine Timestamp DescriptionThe auto-update description currently reads as “auto updated at …” which may be unclear. Consider rephrasing it to “auto-updated at …” with a hyphen to improve readability and conform to stylistic suggestions.
Suggested diff:-This project follows the git-contributor [spec](https://github.com/xudafeng/git-contributor), auto updated at `Tue Apr 01 2025 15:44:10 GMT+0800`. +This project follows the git-contributor [spec](https://github.com/xudafeng/git-contributor), auto-updated at `Tue Apr 01 2025 15:44:10 GMT+0800`.🧰 Tools
🪛 LanguageTool
[uncategorized] ~70-~70: It appears that a hyphen is missing (if ‘auto’ is not used in the context of ‘cars’).
Context: ...//github.com/xudafeng/git-contributor), auto updated atTue Apr 01 2025 15:44:10 GMT+0800....(AUTO_HYPHEN)
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (1)
README.md(1 hunks)
🧰 Additional context used
🪛 LanguageTool
README.md
[uncategorized] ~70-~70: It appears that a hyphen is missing (if ‘auto’ is not used in the context of ‘cars’).
Context: ...//github.com/xudafeng/git-contributor), auto updated at Tue Apr 01 2025 15:44:10 GMT+0800....
(AUTO_HYPHEN)
🪛 markdownlint-cli2 (0.17.2)
README.md
66-66: Images should have alternate text (alt text)
null
(MD045, no-alt-text)
66-66: Images should have alternate text (alt text)
null
(MD045, no-alt-text)
66-66: Images should have alternate text (alt text)
null
(MD045, no-alt-text)
66-66: Images should have alternate text (alt text)
null
(MD045, no-alt-text)
Summary by CodeRabbit