-
-
Notifications
You must be signed in to change notification settings - Fork 949
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
docs(readme.md): add repository badges #161
Conversation
fzn0x
commented
Jan 15, 2022
- provides short information in badges
- clickable badge to opencollective
provides short information in badges clickable badge to opencollective
✔️ Deploy Preview for vigilant-wescoff-04e480 ready! 🔨 Explore the source changes: 6e9b019 🔍 Inspect the deploy log: https://app.netlify.com/sites/vigilant-wescoff-04e480/deploys/61e8c9a83dc1670008fed8c8 😎 Browse the preview: https://deploy-preview-161--vigilant-wescoff-04e480.netlify.app |
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.
NICE
@damienwebdev you are missing that this will also rendered on npm package site! |
a1a9c9e
Which one I should follow? your suggestion or @damienwebdev? or currently we stale it first? |
For me only Merged PRs and contributors can be removed, license should be there for npm |
Maybe we should just link it to the license file |
Do we follow major consensus vote to remove merged PRs and contributors? |
We decided to move this change to v6.1.0, but I still want to outline a conversation that @Shinigami92 and I had in Discord:
The rest seem good. |
Cool, resolving, thanks @damienwebdev and @Shinigami92 😃 |
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.
Lgtm