-
Notifications
You must be signed in to change notification settings - Fork 1.4k
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
🌱 : stop to generate all samples for go/v3 #3275
🌱 : stop to generate all samples for go/v3 #3275
Conversation
/test pull-kubebuilder-e2e-k8s-1-24-7 |
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.
Looks good to me, just one question:
# [Deprecated] - Project version 3 (default) uses plugin go/v3 (default). | ||
scaffold_test_project project-v3 | ||
|
||
# [Currently, default CLI plugin] - [Next version, alpha] Project version v4-alpha | ||
scaffold_test_project project-v4 --plugins="go/v4" | ||
scaffold_test_project project-v4-multigroup --plugins="go/v4" | ||
scaffold_test_project project-v4-declarative-v1 --plugins="go/v4,declarative" |
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.
Should we replicate the -with-deploy-image
and -with-grafana
samples for go/v4?
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.
Forgot to expand the file to see that it already does 🤦 - ignore this
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: camilamacedo86, everettraven The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
/lgtm |
Description
The go/v3 has been deprecated.
Therefore, we no longer need to generate the samples with all addons options for go/v3 and we can keep only project-v3 as a base so we are able to review and compare the scaffolds.
Motivation
Make easier to keep the project maintained