-
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
🌱 cleanup: simplify and consolidate testdata samples #4126
🌱 cleanup: simplify and consolidate testdata samples #4126
Conversation
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: camilamacedo86 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 |
c24bbb5
to
1f2332f
Compare
1f2332f
to
942370a
Compare
942370a
to
f6ee9ab
Compare
f6ee9ab
to
df7dfa6
Compare
df7dfa6
to
ee3fc91
Compare
ee3fc91
to
d8f35b7
Compare
PR needs rebase. 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. |
d8f35b7
to
bca81f3
Compare
I am moving forward with this one. |
The testdata samples are designed to showcase various options for users and enable comprehensive testing of tool commands. By aggregating the Grafana and deploy-image plugins into the multigroup layout, we streamline test coverage and reduce redundancy. This change ensures all options are tested within the multigroup layout while keeping project-v4 with and without the optional plugins, to help us ensure the references for users and keep our CI tests. Now, instead of having a separate sample for each optional plugin, we have: - project-v4 - project-v4-multigroup-with-plugins - project-v4-with-plugins This consolidation reduces the number of samples and tests, making the codebase easier to maintain without sacrificing test coverage or usability for users.
🌱 cleanup: simplify and consolidate testdata samples #4126
The testdata samples are designed to showcase various options for users and enable comprehensive testing of tool commands. By aggregating the Grafana and deploy-image plugins into the multigroup layout, we streamline test coverage and reduce redundancy. This change ensures all options are tested within the multigroup layout while keeping project-v4 with and without the optional plugins, to help us ensure the references for users and keep our CI tests.
Now, instead of having a separate sample for each optional plugin, we have:
This consolidation reduces the number of samples and tests, making the codebase easier to maintain without sacrificing test coverage or usability for users.