-
Notifications
You must be signed in to change notification settings - Fork 36
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
fix(RELEASE-1150) add checks to fieldselectors #583
base: main
Are you sure you want to change the base?
Conversation
Skipping CI for Draft Pull Request. |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #583 +/- ##
========================================
Coverage 84.09% 84.09%
========================================
Files 26 26
Lines 1635 1742 +107
========================================
+ Hits 1375 1465 +90
- Misses 182 192 +10
- Partials 78 85 +7 ☔ View full report in Codecov by Sentry. |
80d21f3
to
cd111ab
Compare
/retest |
/retest |
1 similar comment
/retest |
cd111ab
to
193aa3d
Compare
This commit fixes the issue of FieldSelectors retrurn everything when argument is nill, added checks to each function using fieldselecor. Signed-off-by: Happy Bhati <hbhati@redhat.com>
193aa3d
to
fc70fe4
Compare
New changes are detected. LGTM label has been removed. |
Thanks, @johnbieren, for your approval. @davidmogar Since you reviewed it earlier, please take a look whenever you get a chance. |
This commit fixes the issue of FieldSelectors retrurning everything when the argument is nill, added checks to each function using field selector.