Skip to content

Conversation

jtwile2
Copy link

@jtwile2 jtwile2 commented Sep 29, 2017

Continue reading on corrupted messages in the journal. Addresses Issue #16.

@jtwile2
Copy link
Author

jtwile2 commented Sep 29, 2017

The tests passed, the CI failure is RuboCop issues that were not part of this PRs changes.

@errm
Copy link
Collaborator

errm commented Oct 2, 2017

Thanks @jtwile2 I will certainly take a look at this over the next few days ...

The reevoocop stuff just seems to be related to complexity, I figure we could use a little refactoring at this point so I will take a look at that before merging this in...

@jtwile2
Copy link
Author

jtwile2 commented Oct 2, 2017

Sounds good, I don't have any plans to backport this to the 0.0.x branch, but if you need that, I can take a look.

@errm errm merged commit debe752 into fluent-plugins-nursery:master Oct 3, 2017
chancez pushed a commit to chancez/fluent-plugin-systemd that referenced this pull request Nov 21, 2017
chancez pushed a commit to chancez/fluent-plugin-systemd that referenced this pull request Nov 21, 2017
chancez pushed a commit to chancez/fluent-plugin-systemd that referenced this pull request Nov 21, 2017
chancez pushed a commit to chancez/fluent-plugin-systemd that referenced this pull request Nov 21, 2017
errm added a commit that referenced this pull request May 3, 2018
errm added a commit that referenced this pull request May 3, 2018
This is a reworked solution to #16

Previously addressed by #41... which was removed for licensing reasons.
errm added a commit that referenced this pull request May 3, 2018
This is a reworked solution to #16

Previously addressed by #41... which was removed for licensing reasons.
errm added a commit that referenced this pull request May 3, 2018
This is a reworked solution to #16

Previously addressed by #41... which was removed for licensing reasons
see #60 for background.
@errm errm mentioned this pull request May 3, 2018
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