Skip to content

Adds lift-json-scalaz7 (Scalaz 7 support).#1424

Merged
dpp merged 5 commits into
lift:masterfrom
taylorleese:master
Mar 22, 2013
Merged

Adds lift-json-scalaz7 (Scalaz 7 support).#1424
dpp merged 5 commits into
lift:masterfrom
taylorleese:master

Conversation

@taylorleese

Copy link
Copy Markdown

Adds lift-json-scalaz7 (Scalaz 7 support).

  • Adds a new submodule for lift-json-scalaz7.
  • Maintains lift-json-scalaz for Scalaz 6 support.
  • Updated json-scalaz directory structure from lifweb to liftweb.
  • Updated README to correspond to Scalaz 7.

Discussion on Lift mailing list.

Taylor Leese added 5 commits March 20, 2013 12:43
* All tests pass, but only compiling for Scala 2.10.0 at the moment.
* May need a lift-json-scalaz7 subproject to provide both Scalaz 6 and Scalaz 7 support.
* Adds a new submodule for lift-json-scalaz7.
* Maintains lift-json-scalaz for Scalaz 6 support.
* Updated README to correspond to Scalaz 7.

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

Note, I removed the Kleisli composition example because Validation is not a monad in Scalaz 7 (see the Scalaz 7 thread). Using a for comprehension provides the same functionality albeit with a little more typing.

dpp added a commit that referenced this pull request Mar 22, 2013
Adds lift-json-scalaz7 (Scalaz 7 support). Based on Joni's mailing list thumbs up, I'm applying the patch
@dpp dpp merged commit b1812ab into lift:master Mar 22, 2013
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.

2 participants