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

feat: add explicit check against codeCoverage.expectFrontendCoverageOnly before attempting to collect backend coverage #474

Closed
wants to merge 4 commits into from

Conversation

funkadelic
Copy link

PR for #473. See #473 for details behind the change

…nly before attempting to collect backend coverage
@CLAassistant
Copy link

CLAassistant commented Jun 25, 2021

CLA assistant check
All committers have signed the CLA.

@rsudarson
Copy link

Please release a version with this change. @maartencardinaels @funkadelic

@jogelin
Copy link

jogelin commented Jul 26, 2021

@funkadelic is this PR still planned? My monitoring is showing all of these failing requests...

@funkadelic
Copy link
Author

@maartencardinaels thanks for the PR approval; is there anything else needed from my end to get this merged down to master & released?

@mcaverick
Copy link

@maartencardinaels thanks for the PR approval; is there anything else needed from my end to get this merged down to master & released?

one of the project collaborators will have to merge this PR

@eulersson
Copy link

Could this be revisited? What steps are required to get it through?

@eulersson
Copy link

I forked from master and brought in the same changes @funkadelic has. Therefore the PR I created is an updated duplicate of this PR.

#555

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants