-
Notifications
You must be signed in to change notification settings - Fork 172
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
Remove bots from the contributors list #5233
Remove bots from the contributors list #5233
Conversation
Satishpokala124 is not a collaborator of the repo |
Hi @Satishpokala124, thanks for the contribution! Could you update this to also filter out renovate-bot please? It is currently the bot with the most contributions so I'd like to make sure that it is also filtered out by this change. |
Hi @jmuzina, the renovate-bot is removed using its login. Let me know if that's ok. |
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.
Left a few comments, but this is almost ready to go! :)
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, thanks for the contribution!
@bartaz this is looking good & ready to merge
Thanks @Satishpokala124 for the fix and @jmuzina for review! |
Done
Removes bots from the contributors list
Fixes #5098
QA
Check if PR is ready for release
If this PR contains Vanilla SCSS code changes, it should contain the following changes to make sure it's ready for the release:
Feature 🎁
,Breaking Change 💣
,Bug 🐛
,Documentation 📝
,Maintenance 🔨
.package.json
should be updated relative to the most recent release, following semver convention:Screenshots