Skip to content

Conversation

@dpocock
Copy link

@dpocock dpocock commented Jun 27, 2014

This enables messages to be received from the wire and parsed into Syslog objects.

@gma
Copy link
Owner

gma commented Jul 19, 2014

Cheers for this. Sorry it's taken me a few weeks to get to it – work is mental at the moment.

Comments on commits to follow…

@gma
Copy link
Owner

gma commented Jul 19, 2014

Okay, I think that's all my comments on the commits done.

If you agree with/fancy making any of the suggested changes, could we have a topic branch (e.g. parsing), so that any changes/iterations we make on the branch will show up in the same pull request without us going to/fro on your master branch?

We should also rebase those two merge commits out of the way (I can help with the git for that if necessary) to keep the commit log clean.

Damn I love Python. It's been a while since I've spent more than two minutes looking at some, and it's been fun. :-)

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