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

Loggers -- could we factor them out to use with non-LVish software #104

Open
rrnewton opened this issue Mar 24, 2015 · 4 comments
Open

Loggers -- could we factor them out to use with non-LVish software #104

rrnewton opened this issue Mar 24, 2015 · 4 comments
Assignees
Milestone

Comments

@rrnewton
Copy link
Member

I.e. use them from the IO monad (or possibly a monad transformer) when testing concurrent data structures.

This is somewhat related to #84

CC: @osa1 @alivaramesh

@peter-fogg
Copy link
Contributor

I factored out the logging module and put it up at https://github.com/iu-parfunc/tslogger. There's a branch which depends on the new package here: https://github.com/iu-parfunc/lvars/tree/remove-logger

@rrnewton
Copy link
Member Author

Cool, let's turn that into a pull request, make sure it passes tests, and then merge!

We should have some tests on the tslogger package that make sure its fuzz testing capability is still working. I added this as an issue on the tslogger repo: iu-parfunc/tslogger#2

So as soon as things are up and running well there, and we've merged the forthcoming remove-logger PR, let's close this issue.

@rrnewton rrnewton added this to the 2.0 Release milestone May 10, 2016
@rrnewton
Copy link
Member Author

Ok, we've still got the internal logger. We should merge this. Hopefully that code hasn't been touched so even though it's old it, the merge won't be hard. (CC @DreamLinuxer )

@rrnewton
Copy link
Member Author

Finally merged this branch: d7c4c7b

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

2 participants