-
Notifications
You must be signed in to change notification settings - Fork 84
Correct upstream API References property casing, bump velero docs version #772
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
Conversation
Using psuedo version to include datamover features property. https://pkg.go.dev/github.com/openshift/oadp-operator@v0.0.0-20220722142130-a02487302a02/api/v1alpha1#Features Do this to get psuedo version of latest master commit ``` go get -d github.com/openshift/oadp-operator/api@master go: github.com/openshift/oadp-operator/api@master: github.com/openshift/oadp-operator/api@v0.0.0-20220722142130-a02487302a02: invalid version: missing github.com/openshift/oadp-operator/api/go.mod at revision a024873 ``` Closes openshift#770 openshift#768
Codecov Report
@@ Coverage Diff @@
## master #772 +/- ##
==========================================
- Coverage 36.96% 34.22% -2.75%
==========================================
Files 16 16
Lines 3219 2840 -379
==========================================
- Hits 1190 972 -218
+ Misses 1926 1778 -148
+ Partials 103 90 -13
Continue to review full report at Codecov.
|
|
switching to v0.0.0-20220727202820-c5f87df8fafd for links |
savitharaghunathan
left a comment
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
Corrects API references property fields casing to match what user would type into dpa spec.
Hyperlink to types on pkg.go.dev
Using pseudo version pkg.go.dev to include datamover features property.
https://pkg.go.dev/github.com/openshift/oadp-operator@v0.0.0-20220722142130-a02487302a02/api/v1alpha1#Features
May follow up by replacing pseudo-version with tagged version 1.1.0
This PR is accurate as of a024873
Do this to get psuedo version of latest master commit
Closes #770 #768