-
-
Notifications
You must be signed in to change notification settings - Fork 4k
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
chore: improve README.md layout with badges and colors #10428
base: main
Are you sure you want to change the base?
Conversation
Enhanced the README.md file by adding the npm logo and two new badges: the last commit and contributions badges. These additions help viewers and coders understand the latest commit activity and the number of contributors. Updated the badge colors to better contrast and look more appealing than the previous ones.
The latest updates on your projects. Learn more about Vercel for Git ↗︎
1 Skipped Deployment
|
Co-authored-by: Almeida <github@almeidx.dev>
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.
Sure, but might make sense to add these to all packages
Okay 👍 |
<a href="https://github.com/discordjs/discord.js/commits/main"><img src="https://img.shields.io/github/last-commit/discordjs/discord.js.svg?maxAge=3600&logo=github&logoColor=fff" alt="last commit" /></a> | ||
<a href="https://github.com/discordjs/discord.js/graphs/contributors"><img src="https://img.shields.io/github/contributors/discordjs/discord.js.svg?maxAge=3600&logo=github&logoColor=fff&color=00c7be" alt="contributors" /></a> |
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.
Is it me, or do I feel this is misleading to put in the discord.js package? It would be fine on the root README.md. Imagine if the most recent commit was to brokers... that's not a commit to here.
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.
Yes, but @vladfrangu told me to add it to all packages.
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.
I'd like to see this addressed before merging. Vlad probably didn't realize
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #10428 +/- ##
==========================================
+ Coverage 38.00% 38.03% +0.02%
==========================================
Files 239 239
Lines 15488 15464 -24
Branches 1367 1353 -14
==========================================
- Hits 5886 5881 -5
+ Misses 9587 9568 -19
Partials 15 15
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
Please describe the changes this PR makes and why it should be merged:
This pull request enhances the
README.md
file by introducing the npm logo and two new badges: one for the last commit and one for contributions. These additions provide viewers and contributors with clear information on the latest commit activity and the number of contributors. The badge colors have also been updated for improved contrast and visual appeal, creating a more attractive and informative layout compared to the previous version.Status and versioning classification: