Navigation Menu

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

Tidy the script folder up #3633

Merged
merged 2 commits into from Oct 22, 2021
Merged

Conversation

antgamdia
Copy link
Member

Description of the change

I was about to start fixing the create_release script (#3566) when I noticed this folder was kind of a mess. So this PR is to clean it up a little bit.

Also, I've updated the copyright headers; even if it is something we will have to thoughtfully check at some point, we better start having some uniform text so that it's easier for us later.

Benefits

image
ignore the vs code colors

Possible drawbacks

Even if I did my best checking further file appearances and renaming, perhaps I missed one. It will be something we'll discover when executing the different CI flows. The fix, if happen to be required, should be straighforward.

Applicable issues

Additional information

N/A

Signed-off-by: Antonio Gamez Diaz <agamez@vmware.com>
Signed-off-by: Antonio Gamez Diaz <agamez@vmware.com>
@antgamdia antgamdia changed the title Sort the script folder Tidy the script folder up Oct 21, 2021
@antgamdia
Copy link
Member Author

Merging as the CI got green, but it wasn't reported back to Github (circle ci was experiencing some minor outages yesterday)
cf. https://app.circleci.com/pipelines/github/antgamdia/kubeapps/1358/workflows/7a00e372-f1e9-4bda-b659-ccb2a1e94881

@antgamdia antgamdia merged commit e41ac36 into vmware-tanzu:master Oct 22, 2021
@antgamdia antgamdia deleted the sort-scripts branch October 22, 2021 07:32
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.

None yet

2 participants