Updates to consumer info casing in output

[wallyqs]

Some updates to consumer info command

before:

State:

  Last Delivered Message: Consumer sequence: 1 Stream sequence: 1 Last delivery: 14m42s ago
    Acknowledgment floor: Consumer sequence: 1 Stream sequence: 1 Last Ack: 14m42s ago
        Outstanding Acks: 0 out of maximum 1,000
    Redelivered Messages: 0
    Unprocessed Messages: 0
           Waiting Pulls: 0 of maximum 512

after:

State:

  Last Delivered Message: Consumer sequence: 1 Stream sequence: 1 Last delivery: 14m29s ago
    Acknowledgment Floor: Consumer sequence: 1 Stream sequence: 1 Last ack: 14m29s ago
        Outstanding Acks: 0 out of maximum 1,000
    Redelivered Messages: 0
    Unprocessed Messages: 0
           Waiting Pulls: 0 of maximum 512

[ripienaar]

LGTM

[wallyqs]

I was checking just in case but this silly change got reverted here in a following commit: 2cc0f18#diff-5f4db62e1562fa119614c73fce2b6bc5d8fa55868b7c98cd177c1847a3889158R663

[ripienaar]

Hmm, yours is still in main here:

https://github.com/nats-io/natscli/blame/f77374d50d882f73f4cab62b15f4ce8804c7bb8f/cli/consumer_command.go#L663

and the other one is also in main though harder to see where it went back in, I recall seeing it got undone but too much code changes for a nice fix.

natscli/cli/consumer_command.go

Lines 663 to 665 in f77374d

	cols.AddRowf("Acknowledgment Floor", "Consumer sequence: %s Stream sequence: %s", f(state.AckFloor.Consumer), f(state.AckFloor.Stream))
	} else {
	cols.AddRowf("Acknowledgment Floor", "Consumer sequence: %s Stream sequence: %s Last Ack: %s ago", f(state.AckFloor.Consumer), f(state.AckFloor.Stream), f(sinceRefOrNow(state.TimeStamp, *state.AckFloor.Last)))

[wallyqs]

ah got it, thanks for checking! should have checked main instead… 😅