Reorganize scripts/ folder #6429
Labels
enhancement
good first issue
Good for beginners
help wanted
Features that maintainers are willing to accept but do not have cycles to implement
Comments
yurishkuro
added
good first issue
|
Hey @yurishkuro I'd like to complete this task, does this folders e2e and lint needs to be created on root directory ? |
|
Hey @yurishkuro I have made changes as per your description and here are their screenshot :
If everything seems to be fine then please let me know I'll open PR. |
no, they all should be nested under scripts/ |
One PR per category. |
This was referenced Dec 28, 2024
|
@yurishkuro Fixed DCO in every PR, PTAL. |
|
@Ashu463 you also need to update files referring to those scripts |
This was referenced Dec 31, 2024
yurishkuro
pushed a commit
that referenced
this issue
Dec 31, 2024
## Which problem is this PR solving? - part of #6429 ## Description of the changes - organized build scripts into `scripts/build` --------- Signed-off-by: danish9039 <[email protected]>
This was referenced Dec 31, 2024
Closed
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
The scripts/ folder accumulated many shell scripts that server different purposes. I propose to categorize them better into subfolders.
adaptive-sampling-integration-test.sh->e2e/adaptive-sampling.shbuild-all-in-one-image.sh->build/all-in-one-image.shbuild-crossdock.sh->build/crossdock.shbuild-hotrod-image.sh->build/hotrod-image.shbuild-upload-a-docker-image.sh->build/upload-a-docker-image.shbuild-upload-docker-images.sh->build/upload-docker-images.shcassandra-integration-test.sh->e2e/cassandra.shcheck-go-version.sh->lint/...check-goleak-files.sh->lint/...check-semconv-version.sh->lint/...check-test-files.sh->lint/...clean-binaries.sh->build/...compute-tags.sh->util/...compute-tags.test.sh->util/...dco_check.py->lint/...docker-login.sh->util/...es-integration-test.sh->e2e/elasticsearch.shimport-order-cleanup.py->lint/...kafka-integration-test.sh->e2e/kafka.shpackage-deploy.sh->build/...rebuild-ui.sh->build/...replace_license_headers.py->lint/...spm-integration-test.sh->e2e/spm.shupdate-semconv-version.sh->lint/...updateLicense.py->lint/...(
...means keep the original file name)Approach:
Categories:
e2e/*lint/*build/*The text was updated successfully, but these errors were encountered: