Skip to content

Conversation

aidecoe
Copy link
Contributor

@aidecoe aidecoe commented Dec 7, 2017

This is where final result is evaluated.

This is where final result is evaluated.
Copy link
Collaborator

@GuillaumeSeren GuillaumeSeren left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Copy link
Member

@flokli flokli left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@aidecoe As stated in #173, it's hard to verify things work (and keep doing so) without tests, or even example messages to validate against manually. So please provide some example messages.

@GuillaumeSeren
Copy link
Collaborator

Merged closing 🍺

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.

3 participants