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

Add a CLI option to disable the timestamp in the prefix #219

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

stack-head
Copy link

Hey, this project recently helped me out, thanks for publishing it. I'm running the same application on two different systems, and it's a bit easier to compare the logs if there's no timestamp on the messages. It's a pretty simple change but, let me know if you have any feedback.

@codecov-io
Copy link

codecov-io commented Apr 19, 2020

Codecov Report

Merging #219 into master will decrease coverage by 0.16%.
The diff coverage is 50.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #219      +/-   ##
==========================================
- Coverage   49.16%   49.00%   -0.17%     
==========================================
  Files          11       11              
  Lines         598      602       +4     
  Branches       88       89       +1     
==========================================
+ Hits          294      295       +1     
- Misses        293      295       +2     
- Partials       11       12       +1     
Impacted Files Coverage Δ
honcho/command.py 0.00% <0.00%> (ø)
honcho/printer.py 80.85% <60.00%> (-3.24%) ⬇️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 19c7890...02982d6. Read the comment docs.

@stack-head
Copy link
Author

Owner: Is there anything I can do to get this merged? I'm interested in sharing the option with collaborators, would be easier if it can get merged into the next release.

@stack-head
Copy link
Author

Hey maintainer, am still interested in adding this option, if you have the time.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants