Skip to content
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: improve cluster list formatting #119

Merged
merged 2 commits into from
Aug 25, 2019
Merged

fix: improve cluster list formatting #119

merged 2 commits into from
Aug 25, 2019

Conversation

amcgee
Copy link
Member

@amcgee amcgee commented Aug 24, 2019

Fixes a minor bug where Up about a minute didn't match the Up \d+ regex. Also passes -a to docker ps so that we can also see stopped containers, and adds a status check for Exited

@amcgee amcgee requested a review from varl August 24, 2019 15:10
Copy link
Contributor

@varl varl left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Neat. LGTM.

@amcgee amcgee merged commit 8b6411c into master Aug 25, 2019
dhis2-bot pushed a commit that referenced this pull request Aug 25, 2019
## [2.1.1](v2.1.0...v2.1.1) (2019-08-25)

### Bug Fixes

* improve cluster list formatting ([#119](#119)) ([8b6411c](8b6411c))
@dhis2-bot
Copy link
Contributor

@varl varl deleted the fix/cluster-up-chalk branch August 25, 2019 13:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants