-
-
Notifications
You must be signed in to change notification settings - Fork 368
Add new sponsor to README #3320
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
🥷 Code experts: jjw24, VictoriousRaptor jjw24 has most 👩💻 activity in the files. See details
Activity based on git-commit:
Knowledge based on git-blame: To learn more about /:\ gitStream - Visit our Docs |
@check-spelling-bot Report🔴 Please reviewSee the 📂 files view, the 📜action log, or 📝 job summary for details.
See ❌ Event descriptions for more information. Using Pattern suggestions ✂️ (1)You could add these patterns to
If the flagged items are 🤯 false positivesIf items relate to a ...
|
Be a legend 🏆 by adding a before and after screenshot of the changes you made, especially if they are around UI/UX. |
📝 WalkthroughWalkthroughThe README.md file has been updated to include an additional sponsor link and logo. The changes insert a new anchor tag linking to a GitHub profile with an associated image tag for the user's avatar. Line breaks were adjusted to maintain consistent formatting in the sponsorship section. No modifications were made to exported entities. Changes
Possibly related PRs
Suggested reviewers
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 (1)
README.md (1)
342-344
: Accessibility Improvement: Add Alt Attribute to New Sponsor ImageThe new sponsor link for AjmalParkar006 is correctly added. However, the
<img>
tag lacks analt
attribute, which is important for accessibility and to satisfy markdown lint rules (MD045). Consider adding descriptive alt text (e.g.,"Ajmal Parkar Sponsor Logo"
).Diff suggestion:
- <a href="/AjmalParkar006"> - <img src='https://avatars.githubusercontent.com/u/76547256?v=4' width="10%"/> - </a> + <a href="/AjmalParkar006"> + <img src='https://avatars.githubusercontent.com/u/76547256?v=4' width="10%" alt="Ajmal Parkar Sponsor Logo"/> + </a>🧰 Tools
🪛 markdownlint-cli2 (0.17.2)
343-343: Images should have alternate text (alt text)
null(MD045, no-alt-text)
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (1)
README.md
(1 hunks)
🧰 Additional context used
🪛 markdownlint-cli2 (0.17.2)
README.md
340-340: Images should have alternate text (alt text)
null
(MD045, no-alt-text)
343-343: Images should have alternate text (alt text)
null
(MD045, no-alt-text)
⏰ Context from checks skipped due to timeout of 90000ms (4)
- GitHub Check: gitStream workflow automation
- GitHub Check: gitStream.cm
- GitHub Check: gitStream.cm
- GitHub Check: gitStream.cm
🔇 Additional comments (1)
README.md (1)
337-338
: Formatting Enhancement: Additional Line BreaksThe insertion of two
<br />
tags helps visually separate the sponsor logos and maintains an organized layout. Please verify that this extra spacing renders well across various devices and screen sizes.
Wasn't sure where to put the new sponsor, if I placed them incorrectly, feel free to correct me.