Skip to content
This repository has been archived by the owner on Sep 7, 2022. It is now read-only.

Further testing of case classes #2

Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

Further testing of case classes #2

wants to merge 4 commits into from

Conversation

rickb777
Copy link

@rickb777 rickb777 commented May 3, 2013

I added some extra test cases for case classes containing optional collections, which is a common usage pattern for us.

@jasongoodwin
Copy link

Hello there good sir.
Thanks for your pull request. We've started doing a bit of work to the fork - I'll merge in the changes at a slightly later point in time. I plan on spend some free time shortly looking at the project in relation to scala 2.10's API changes to see if we can keep refreshing this.
I'll drop another note once the request is complete.

Thanks again,

  • Jason

Resolved Conflicts:
	src/main/scala/com/codahale/jerkson/util/CaseClassSigParser.scala
	src/test/scala/com/codahale/jerkson/tests/CaseClassSupportSpec.scala

Changed name in build.sbt to include the repo name (rickb777).
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants