Skip to content
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

chore: apiextensions.k8s.io/v1 #1048

Closed
wants to merge 1 commit into from
Closed

Conversation

jsoref
Copy link
Member

@jsoref jsoref commented Feb 2, 2021

Signed-off-by: Josh Soref [email protected]

Checklist:

This PR is because argoproj/argo-helm#525 and argoproj/argo-helm#563 and argoproj/argo-helm#565 all fail because of this.

@jsoref
Copy link
Member Author

jsoref commented Feb 2, 2021

I suspect that the current failure would require something like @alexmt 's argoproj/argo-cd#1773

@whynowy
Copy link
Member

whynowy commented Feb 2, 2021

What you should do is updating https://github.com/argoproj/argo-events/blob/master/hack/crdgen.sh instead of making changes to generated files directly.

@whynowy
Copy link
Member

whynowy commented Feb 6, 2021

@jsoref - would you like to continue working on this or me to take it over?

@jsoref
Copy link
Member Author

jsoref commented Feb 7, 2021

@whynowy Please be my guest

@whynowy
Copy link
Member

whynowy commented Feb 8, 2021

#1054

@whynowy whynowy closed this Feb 9, 2021
@jsoref jsoref deleted the lint-v1 branch February 9, 2021 21:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants