-
Notifications
You must be signed in to change notification settings - Fork 351
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
WX-1420 Fix GCP Batch label regex restriction #7355
Conversation
scalafmt |
Fixed that by removing the test case. Now unit tests passed but the codecov failed. |
ah, nice FWIW |
Codecov isn't a mandatory check, we just don't have a way to tell Github to display the number without a red or green check. |
Cool, so pending one more approval and then merge? (also finger cross on other tests) |
Since none of the functional code changed from #7353 which has two approvals, I will wait for tests to pass and admin-merge. |
Thanks for your contribution! |
Co-authored-by: Beibei Chen <beibei@formbio.com> Co-authored-by: Janet Gainer-Dewar <jdewar@broadinstitute.org>
Clone of #7353 to run tests