Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

remove Event.String()? #25

Closed
nathany opened this issue Jul 8, 2014 · 1 comment
Closed

remove Event.String()? #25

nathany opened this issue Jul 8, 2014 · 1 comment
Assignees

Comments

@nathany
Copy link
Contributor

nathany commented Jul 8, 2014

The specifics of how this string is formatted doesn't seem like it belongs in the library.

Replace String() with a helper function in the example and test suite.

@nathany nathany added the API label Jul 8, 2014
@nathany nathany self-assigned this Jul 8, 2014
@nathany nathany changed the title remove Event.String() remove Event.String()? Jul 10, 2014
@nathany
Copy link
Contributor Author

nathany commented Aug 16, 2014

Leaving this in for now as a debugging aid. it doesn't prevent anyone from writing a utility function to represent Event in another way.

@nathany nathany closed this as completed Aug 16, 2014
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

No branches or pull requests

1 participant