You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Should we implement a hard CI check that requires a net positive code coverage on the diff? This would help ensure we are writing decent tests but I could also see it being a bit of a nuisance
The text was updated successfully, but these errors were encountered:
I think it should be implemented, but ignored when it makes sense. For example, a style-only change that touches several modules should not be required to have net positive code coverage, but new feature additions should.
Should we implement a hard CI check that requires a net positive code coverage on the diff? This would help ensure we are writing decent tests but I could also see it being a bit of a nuisance
The text was updated successfully, but these errors were encountered: