-
-
Notifications
You must be signed in to change notification settings - Fork 382
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
fix(cmd): disable cobra default completion cmd #1117
fix(cmd): disable cobra default completion cmd #1117
Conversation
CLA Assistant Lite bot All contributors have signed the CLA ✍️ ✅ |
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.
Thank you and congratulations 🎉 for opening your very first pull request in keploy
Apply Sweep Rules to your PR?
|
I have read the CLA Document and I hereby sign the CLA |
Hey @dreamjz the PR is correct and fixes the issue. Could you just complete the DCO check before I merge it? You can see how to pass it by clicking on the Details button. |
* added automatic portChange feature if current port is busy Signed-off-by: EraKin575 <tejaskumar574@gmail.com> * added for for continuous port change if consecutive ports are busy Signed-off-by: EraKin575 <tejaskumar574@gmail.com> * defined a upper and lower bound and maxatempts Signed-off-by: EraKin575 <tejaskumar574@gmail.com> * added a case where the system can't find a available port Signed-off-by: EraKin575 <tejaskumar574@gmail.com> * improved code indentation Signed-off-by: EraKin575 <tejaskumar574@gmail.com> * fixed emoji Signed-off-by: EraKin575 <tejaskumar574@gmail.com> * fixed logger Signed-off-by: EraKin575 <tejaskumar574@gmail.com> * removed useless imports Signed-off-by: EraKin575 <tejaskumar574@gmail.com> * removed unnecessary logs Signed-off-by: EraKin575 <tejaskumar574@gmail.com> --------- Signed-off-by: EraKin575 <tejaskumar574@gmail.com> Signed-off-by: dreamjz <25699818+dreamjz@users.noreply.github.com>
Signed-off-by: dreamjz <25699818+dreamjz@users.noreply.github.com>
47be373
to
2127c13
Compare
OK |
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.
LGTM
Related Issue
Closes: #1116
Describe the changes you've made
Disable cobra default completion command
Type of change
Please let us know if any test cases are added
NA
Describe if there is any unusual behaviour of your code(Write
NA
if there isn't)NA
Checklist:
Screenshots (if any)