-
Notifications
You must be signed in to change notification settings - Fork 78
Adding static-only switch to pickling #147
Conversation
|
Odd, this builds and tests run successfully on my machine... |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Let's move those to src/test/scala, since they're test-only.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yeah, that's what I thought. I left it there though because I was following what you guys did in async – I was wondering why this stuff wasn't in test for you...
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Async has it in src/test/scala though: https://github.com/scala/async/blob/master/src/test/scala/scala/async/package.scala
|
LGTM |
Adding static-only switch to pickling
Adding static-only switch to pickling
Adding static-only switch to pickling
No description provided.