-
Notifications
You must be signed in to change notification settings - Fork 1.8k
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 a few links in the development file under ... #4256
Changes from 1 commit
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -471,14 +471,14 @@ If you need to add a new CRD type, you will need to add: | |
- [clusterrole-aggregate-edit.yaml](./config/clusterrole-aggregate-edit.yaml) | ||
- [clusterrole-aggregate-view.yaml](./config/clusterrole-aggregate-view.yaml) | ||
1. Add go structs for the types in | ||
[pkg/apis/pipeline/v1alpha1](./pkg/apis/pipeline/v1alpha1) e.g | ||
[condition_types.go](./pkg/apis/pipeline/v1alpha1/condition_types.go) This | ||
[pkg/apis/pipeline/v1beta1](./pkg/apis/pipeline/v1beta1) e.g | ||
[condition_types.go](./pkg/apis/pipeline/v1beta1/condition_types.go) This | ||
should implement the | ||
[Defaultable](./pkg/apis/pipeline/v1alpha1/condition_defaults.go) and | ||
[Validatable](./pkg/apis/pipeline/v1alpha1/condition_validation.go) | ||
[Defaultable](./pkg/apis/pipeline/v1beta1/condition_defaults.go) and | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. similarly, here please replace And the following line, There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Thank you for reviewing @pritidesai ! |
||
[Validatable](./pkg/apis/pipeline/v1beta1/condition_validation.go) | ||
interfaces as they are needed for the webhook in the next step. | ||
1. Register it with the [webhook](./cmd/webhook/main.go) | ||
1. Add the new type to the | ||
[list of known types](./pkg/apis/pipeline/v1alpha1/register.go) | ||
[list of known types](./pkg/apis/pipeline/v1beta1/register.go) | ||
|
||
_See [the API compatibility policy](api_compatibility_policy.md)._ |
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.
please change
condition_types.go
totask_types.go
, conditions are deprecated and do not exist in thebeta
API.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.
Thank you for reviewing @pritidesai !
I committed : f71a32c