-
Notifications
You must be signed in to change notification settings - Fork 62
📖 Update GoDoc comments for the ClusterExtension
API
#1158
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
📖 Update GoDoc comments for the ClusterExtension
API
#1158
Conversation
✅ Deploy Preview for olmv1 ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #1158 +/- ##
=======================================
Coverage 75.45% 75.45%
=======================================
Files 35 35
Lines 1919 1919
=======================================
Hits 1448 1448
Misses 329 329
Partials 142 142
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
There are a couple PRs that are in the merge queue that will require a rebase for these changes. Adding a hold for now /hold |
Signed-off-by: everettraven <[email protected]>
Signed-off-by: everettraven <[email protected]>
Signed-off-by: everettraven <[email protected]>
Signed-off-by: everettraven <[email protected]>
4c3fb5e
to
3814fa0
Compare
Signed-off-by: everettraven <[email protected]>
Rebased on latest merged API changes. unholding /hold cancel |
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
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
21670ae
Description
Updates the GoDoc comments in the
ClusterExtension
API to be more user readable, user the JSON field name in references, and ensures all optional fields are marked with the//+optional
comment tag// +optional
tag #1147Reviewer Checklist