-
Notifications
You must be signed in to change notification settings - Fork 14
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
Cast deps to empty #214
Cast deps to empty #214
Conversation
72638c0
to
0932047
Compare
0932047
to
eaf5054
Compare
Codecov Report
@@ Coverage Diff @@
## custom_queries #214 +/- ##
==================================================
- Coverage 88.02% 87.78% -0.25%
==================================================
Files 25 24 -1
Lines 1278 1302 +24
==================================================
+ Hits 1125 1143 +18
- Misses 153 159 +6
📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more |
eaf5054
to
c137a88
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Implementation seems correct, but the critical test is missing.
Also, one styling comment - you decide if you want to follow it.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
closes #201