-
Notifications
You must be signed in to change notification settings - Fork 206
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
Support resource types that aren't quite "really" resources #3525
Conversation
5eb501f
to
45cf1a3
Compare
This is related to #3398 |
/ok-to-test sha=45cf1a3 |
v2/tools/generator/internal/codegen/embeddedresources/remove_empty_objects.go
Outdated
Show resolved
Hide resolved
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.
Your check for top level Spec and Status types is flawed, but otherwise this is good.
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.
Removing my Request Changes so you're not blocked addressing this before I'm available tomorrow morning.
45cf1a3
to
e2b484c
Compare
Codecov ReportAttention:
Additional details and impacted files@@ Coverage Diff @@
## main #3525 +/- ##
==========================================
+ Coverage 53.10% 53.12% +0.01%
==========================================
Files 1286 1286
Lines 402000 402010 +10
==========================================
+ Hits 213496 213570 +74
+ Misses 157888 157828 -60
+ Partials 30616 30612 -4 ☔ View full report in Codecov by Sentry. |
/ok-to-test sha=e2b484c |
They have a HEAD rather than a GET and no body parameters
e2b484c
to
88a5e04
Compare
/ok-to-test sha=88a5e04 |
They have a HEAD rather than a GET and no body parameters
If applicable: