Closed dharmit closed 2 years ago
[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To complete the pull request process, please assign kadel after the PR has been reviewed.
You can assign the PR to them by writing /assign @kadel
in a comment when ready.
The full list of commands accepted by this bot can be found here.
@kadel @mohammedzee1000 how to take PRs further on this repo in absence of Travis CI?
@kadel @mohammedzee1000 how to take PRs further on this repo in absence of Travis CI?
Probably question for @anandrkskd or @prietyc123
This was supposed to be addressed more than 7 months ago https://github.com/openshift/odo/issues/4315
If we don't have working automated tests you will have to test this manually.
Issues go stale after 90d of inactivity.
Mark the issue as fresh by commenting /remove-lifecycle stale
.
Stale issues rot after an additional 30d of inactivity and eventually close.
Exclude this issue from closing by commenting /lifecycle frozen
.
If this issue is safe to close now please do so with /close
.
/lifecycle stale
Stale issues rot after 30d of inactivity.
Mark the issue as fresh by commenting /remove-lifecycle rotten
.
Rotten issues close after an additional 30d of inactivity.
Exclude this issue from closing by commenting /lifecycle frozen
.
If this issue is safe to close now please do so with /close
.
/lifecycle rotten /remove-lifecycle stale
@dharmit: PR needs rebase.
Some minor cosmetic changes that I found worth doing while going through the code:
s/Odo/odo
at one place.s/exit/eixst
at one place.-n
instead of! -z
based on https://github.com/koalaman/shellcheck/wiki/SC2236 (auto suggested by GoLand)