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
Addition of version argument to installation script #1640
base: main
Are you sure you want to change the base?
Addition of version argument to installation script #1640
Conversation
Signed-off-by: AhmedLotfy02 <[email protected]>
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.
we should also add some docs about this usage in the docs repo.
keploy.sh
Outdated
VERSION_TAG="latest" | ||
else | ||
# If specific version is provided, use that version | ||
VERSION_TAG="v$VERSION" |
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.
I dont think we need to append a v
to the version. Please remove that.
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.
@slayerjain Done
keploy.sh
Outdated
install_keploy_arm() { | ||
curl --silent --location "https://github.com/keploy/keploy/releases/latest/download/keploy_linux_arm64.tar.gz" | tar xz -C /tmp | ||
curl --silent --location "https://github.com/keploy/keploy/releases/download/$VERSION_TAG/keploy_linux_arm64.tar.gz" | tar xz -C /tmp |
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.
Why have you only updated this in the arm section? There are somary other places we will need to update it.
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.
@slayerjain Done adding it to amd section and docker section.
Adding DCO check message . Signed-off-by: Tushar kalsi <[email protected]>
Signed-off-by: AhmedLotfy02 <[email protected]>
Signed-off-by: AhmedLotfy02 <[email protected]>
Signed-off-by: AhmedLotfy02 <[email protected]>
Signed-off-by: AhmedLotfy02 <[email protected]>
Signed-off-by: AhmedLotfy02 <[email protected]>
@slayerjain All edits requested are done. |
…eploy#1640 Signed-off-by: Tushar kalsi <[email protected]>
Signed-off-by: Tushar kalsi <[email protected]>
@AhmedLotfy02 Please resolve the merge conflicts. |
@gouravkrosx Done. |
@AhmedLotfy02 please resolve the merge conflicts in Readme file |
@shivamsouravjha Done. |
Related Issue
Closes: #1639
Describe the changes you've made
Addition of version argument to installation script
Type of change
Please let us know if any test cases are added
Please describe the tests(if any). Provide instructions how its affecting the coverage.
Describe if there is any unusual behaviour of your code(Write
NA
if there isn't)A clear and concise description of it.
Checklist:
Screenshots (if any)