search

syndesisio / syndesis.io

The syndesis.io Website
https://syndesis.io
10 stars 34 forks source link

changing oc get pods command to watch pods #61

Closed samueltauil closed 7 years ago

zregvart commented 7 years ago

I was wondering about that, I'm sure that @jimmidyson and @chirino know about -w switch, but perhaps wanted a nicer UI?

samueltauil commented 7 years ago

just added the -w switch because it can run across different os platforms as it is part of oc binary.

zregvart commented 7 years ago

@samueltauil to me that makes absolute sense, I just wandered why @jimmidyson and @chirino thought this was better 😸

chirino commented 7 years ago

Lgtm

rhuss commented 7 years ago

personally I prefer plain 'watch' as it is more dynamic and cleans up the history stuff. What's about adding both examples ?

jimmidyson commented 7 years ago

-w is a bit hard to follow imo, hence why i preferred watch. Happy to have both as examples, but :-1: to just -w

pure-bot[bot] commented 7 years ago

Pull request approved by @zregvart - applying approved label