-
Notifications
You must be signed in to change notification settings - Fork 18.6k
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 logs, so now, old and followed logs has same format without [] #10424
Conversation
Hah, sorry for that :) |
can you please make sure there are no examples in the docs that might also change? |
@SvenDowideit Yup, okay. |
Hah, pretty strange :) seems like Fixed format is not so Fixed |
Signed-off-by: Alexander Morozov <lk4d4@docker.com>
ping @jfrazelle |
@@ -45,23 +45,23 @@ After running docker events a container 786d698004576 is started and stopped | |||
(The container name has been shortened in the output below): |
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.
If you're going to change the container ID to long form, can you please remove the above?
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.
Oh, of course :) It's not me, seems like was changed in code at some time.
Or seems like I got it wrong, I'll shorten them back :/
Signed-off-by: Alexandr Morozov <lk4d4@docker.com>
@SvenDowideit Fixed |
Docs LGTM |
LGTM |
LGTM |
Fix logs, so now, old and followed logs has same format without []
Also, this shows us how often this feature used :)