-
Notifications
You must be signed in to change notification settings - Fork 616
feat: set warning status for deprecated fields #7797
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
base: main
Are you sure you want to change the base?
Conversation
56741e7 to
790bb0c
Compare
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #7797 +/- ##
==========================================
+ Coverage 72.64% 72.67% +0.02%
==========================================
Files 235 235
Lines 34873 34938 +65
==========================================
+ Hits 25334 25390 +56
- Misses 7736 7743 +7
- Partials 1803 1805 +2 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
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, thanks!
790bb0c to
c91527d
Compare
|
@jukie thanks for pointing that out, I was in the middle of a refactor, didn't give it a good look. Can you review this now? |
10f84d0 to
4298f48
Compare
Signed-off-by: Rudrakh Panigrahi <rudrakh97@gmail.com>
4298f48 to
1b5902e
Compare
|
/retest |
What type of PR is this?
feat: set warning status for deprecated fields
What this PR does / why we need it:
Introduces a warning status condition for xPolicy resources if deprecated fields are used. This will be helpful in highlighting usage of these deprecated fields and can also be used for other such issues that need to be highlighted by EG.
Which issue(s) this PR fixes:
Related #6717
Release Notes: Yes