-
Notifications
You must be signed in to change notification settings - Fork 104
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
Sort output of plan status and history commands #1371
Conversation
9afb022
to
995e005
Compare
995e005
to
4e1b222
Compare
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.
Nice, I'm looking forward to have this in :)
cdd8ee2
to
ae9d3fd
Compare
ae9d3fd
to
4522f08
Compare
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.
Still good, I wouldn't mind having a temporary variable in the plan_history.go though.
4522f08
to
2870531
Compare
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!
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!
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.
this needs a test in plan_status_test.go
part of requesting a test is to ensure that it is testable... with a timestamp formatted output it isn't clear.
Alphabetise (by plan) the output of the plan status and history commands so that they're consistent with each run. This helps human operators when monitoring a deployment. Signed-off-by: Nick Jones <nick@dischord.org>
2870531
to
e287881
Compare
Include timestamps in output, and define more than one plan. Signed-off-by: Nick Jones <nick@dischord.org>
a1cc0e2
to
e1bcebb
Compare
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
* Sort output of plan status and plan history commands Alphabetise (by plan) the output of the plan status and history commands so that they're consistent with each run. This helps human operators when monitoring a deployment. Signed-off-by: Nick Jones <nick@dischord.org> Signed-off-by: Thomas Runyon <runyontr@gmail.com>
What this PR does / why we need it:
Alphabetise (by plan) the output of the plan status and history commands so that they're consistent with each run. This helps human operators when monitoring a deployment.
Fixes #1354