Skip to content
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

Adding badges for JetBrains Plugin Ratings, run [JetBrainsRating]. #4897 #4898

Merged
merged 3 commits into from
Apr 12, 2020
Merged

Adding badges for JetBrains Plugin Ratings, run [JetBrainsRating]. #4897 #4898

merged 3 commits into from
Apr 12, 2020

Conversation

ChrisCarini
Copy link
Contributor

Hello! Please review this PR - first time doing node development, but followed existing JetBrains badge patterns among other badges. Added tests and tested via the local web interface.

@shields-ci
Copy link

shields-ci commented Apr 11, 2020

Messages
📖 ✨ Thanks for your contribution to Shields, @ChrisCarini!

Generated by 🚫 dangerJS against 739509b

@shields-cd shields-cd temporarily deployed to shields-staging-pr-4898 April 11, 2020 19:45 Inactive
Copy link
Member

@chris48s chris48s left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for working on this. I've left a couple of minor comments inline

services/jetbrains/jetbrains-rating.service.js Outdated Show resolved Hide resolved
services/jetbrains/jetbrains-rating.tester.js Outdated Show resolved Hide resolved
@chris48s chris48s added the service-badge New or updated service badge label Apr 11, 2020
@calebcartwright calebcartwright changed the title Adding badges for JetBrains Plugin Ratings. #4897 Adding badges for JetBrains Plugin Ratings, run [JetBrainsRating]. #4897 Apr 11, 2020
@shields-cd shields-cd temporarily deployed to shields-staging-pr-4898 April 12, 2020 08:01 Inactive
Copy link
Member

@chris48s chris48s left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM - thanks 👍

@repo-ranger repo-ranger bot merged commit 2bf6dfd into badges:master Apr 12, 2020
@ChrisCarini
Copy link
Contributor Author

Awesome, thank you, @chris48s !!

I also saw in the contribution docs that the @shield-deployment bot should have come along with a message - but it seems that failed. I'm assuming that since this change was merged into master it'll go out with the next deploy, correct? :)

@calebcartwright
Copy link
Member

I'm assuming that since this change was merged into master it'll go out with the next deploy, correct?

Correct. Sometimes the deployment bot message just doesn't get posted for whatever reason

@ChrisCarini
Copy link
Contributor Author

Ok cool - I saw a few others where the deployment bot also didn't post. Looking forward to seeing this out!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
service-badge New or updated service badge
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants