-
Notifications
You must be signed in to change notification settings - Fork 493
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
Manage our tools' dependencies in a structured way #3318
Comments
/priority important-longterm |
@shaneutt: GuidelinesPlease ensure that the issue body includes answers to the following questions:
For more details on the requirements of such an issue, please see here and ensure that they are met. If this request no longer meets these requirements, the label can be removed In response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. |
Why not both? I'd suggest the following approach:
In order to keep a unified interface for installing and running the tools we can leverage something like We follow a similar approach in |
Yep, this approach makes a lot of sense to me. Just want to check with the other maintainers to see if they are on board |
I am not opposed to us giving this a try, I do think we'll want to document it perhaps a bit "excessively". |
Agreed, it seems a little Rube-Goldberg-y to me, but seems worth trying as well. |
What would you like to be added:
We have many dependencies in our hack script that often do not get updated. A non-exhaustive list follows:
gateway-api/hack/verify-crds-kind.sh
Lines 48 to 49 in cb846ce
gateway-api/hack/verify-golint.sh
Lines 21 to 22 in cb846ce
gateway-api/hack/verify-yamllint.sh
Lines 21 to 22 in cb846ce
We need to discover all these dependencies and set up a way to either
The text was updated successfully, but these errors were encountered: