-
Notifications
You must be signed in to change notification settings - Fork 146
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
Coverage integration into PR workflow #371
Comments
@geemus @plribeiro3000 I’ve requested codecov.io integration for If you think that’s too much - no worries, let me know and I'll figure something out. We cannot use coveralls since they don’t support merged reports well and we need to do parallel testing (whole test suite takes almost 3 hours if we run sequentially). Apologies for bothering you folks m(_ _)m EDIT: removed the YAML instructions since we don't want the whole org to use our bot (d'oh). |
@Temikus I believe it should be approved now, just let me know if you need anything else. Thanks! |
👍 |
@geemus Thank you, Sir! |
Looks like the bot is starting to do its' magic: #373 I'll close this out once I'm sure everything's working (this particular PR doesn't have a base due to being submitted before the bot was approved) |
This seems to be working, closing! |
Bringing out an item from #370 here, since this is pending Fog org action.
fog-google-bot
permissions to display coveragemaster
branch has coverageThe text was updated successfully, but these errors were encountered: