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.

Sign up for GitHub

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

Jump to bottom

Delete extraneous files #1615

Closed
wants to merge 3 commits into from
Closed

Delete extraneous files #1615

wants to merge 3 commits into from

Conversation

Alan-Cha
Copy link
Member

No description provided.

@Alan-Cha Alan-Cha requested a review from kalantar August 31, 2023 11:59
@Alan-Cha
Copy link
Member Author

@kalantar I believe you also mentioned that there were additional /testdata files that you thought may need cleaning up. Maybe the ones under /testdata/controllers?

@Alan-Cha
Bump version
712e88c 
Signed-off-by: Alan Cha <[email protected]>
@kalantar
Copy link
Member

kalantar commented Sep 1, 2023

Also can remove:

  • base/summarymetrics
  • charts/litmuschaos
  • driver/filedriver*
  • testdata/controllers - keep only garb.age and config.yaml
  • ./config.yaml ??

@kalantar kalantar added the cncf label Sep 1, 2023
@kalantar kalantar mentioned this pull request Sep 5, 2023
Open
14 tasks
@Alan-Cha Alan-Cha mentioned this pull request Sep 5, 2023
Merged
@Alan-Cha
Copy link
Member Author

Alan-Cha commented Sep 6, 2023

Replaced by #1622

@Alan-Cha Alan-Cha closed this Sep 6, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@kalantar kalantar Awaiting requested review from kalantar

Assignees

@Alan-Cha Alan-Cha

Labels
cncf
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@Alan-Cha @kalantar