mistercrunch commented on code in PR #33593:
URL: https://github.com/apache/superset/pull/33593#discussion_r2115054684
##########
.github/workflows/superset-frontend.yml:
##########
@@ -121,24 +121,6 @@ jobs:
files: merged-output/coverage-summary.json
slug: apache/superset
- core-cover:
Review Comment:
NOTE: this forced 100% test coverage across `core/`. Dropped it as we're
bringing more things in here. Could be nice to keep the 100% requirement on
parts of it, but gets in the way of velocity. Test now covered as part of the
main repo tests. We had double test coverage here before which is a bit
wasteful.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
