Skip to content

Conversation

@evalphobia
Copy link
Owner

re-implement #3 and refactored testing for persistent connection.

  • Add New() function to create hook with persistent fluentd logger
    • it could use same connection
  • Change testing to check the message for both New() and NewHook()
  • Change hook struct from unimported to imported
    • Allow user to customize fluentd logger

@coveralls
Copy link

Coverage Status

Coverage increased (+1.0%) to 84.431% when pulling 2663255 on feature/persistent-conn into 9344206 on master.

1 similar comment
@coveralls
Copy link

coveralls commented Aug 27, 2016

Coverage Status

Coverage increased (+1.0%) to 84.431% when pulling 2663255 on feature/persistent-conn into 9344206 on master.

@evalphobia evalphobia force-pushed the feature/persistent-conn branch from 2663255 to 1b4c20a Compare August 27, 2016 15:26
@coveralls
Copy link

coveralls commented Aug 27, 2016

Coverage Status

Coverage increased (+4.0%) to 87.425% when pulling 1b4c20a on feature/persistent-conn into 9344206 on master.

@evalphobia evalphobia merged commit d7c98fb into master Aug 27, 2016
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