Skip to content

Conversation

mmaslowskicc
Copy link

No description provided.

@coveralls
Copy link

coveralls commented Aug 1, 2016

Coverage Status

Coverage increased (+0.2%) to 92.647% when pulling 581ef5c on mmaslowskicc:log-streams-limit into 021691b on jorgebastida:master.

@jorgebastida
Copy link
Owner

I do like this idea, but I think the code would benefit from a few changes:

  • Handle thread termination when the program has to exit.
  • Share a deque to avoid duplicated log lines between generators?

If anybody is happy to jump into updating this PR, I'll happily merge it.

@jorgebastida jorgebastida added this to the 0.12 milestone Apr 12, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants