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

Modularize play-json / play-functional / play-datacommons #754

Merged
merged 9 commits into from
Feb 21, 2013
Merged

Modularize play-json / play-functional / play-datacommons #754

merged 9 commits into from
Feb 21, 2013

Conversation

mandubian
Copy link
Contributor

No description provided.

jeantil and others added 4 commits February 16, 2013 00:11
This module is the first dependency which needs to be extracted to extract play-json.
Second dependency which has to be extracted to be able to make play-json standalone
…k to play core project because I want to keep Forms as is for now
@mandubian mandubian closed this Feb 19, 2013
jeantil and others added 5 commits February 19, 2013 16:32
This module is the first dependency which needs to be extracted to extract play-json.
Second dependency which has to be extracted to be able to make play-json standalone
…k to play core project because I want to keep Forms as is for now
@mandubian mandubian reopened this Feb 19, 2013
mcalmels pushed a commit to mcalmels/Play20 that referenced this pull request Feb 19, 2013
mcalmels pushed a commit to mcalmels/Play20 that referenced this pull request Feb 19, 2013
@jto
Copy link

jto commented Feb 19, 2013

+1

guillaumebort added a commit that referenced this pull request Feb 21, 2013
Modularize play-json / play-functional / play-datacommons
@guillaumebort guillaumebort merged commit dfa557a into playframework:master Feb 21, 2013
publishArtifact in (Compile, packageSrc) := true
)
).settings(com.typesafe.sbtscalariform.ScalariformPlugin.defaultScalariformSettings: _*)
.dependsOn(IterateesProject, FunctionalProject, DataCommonsProject)
Copy link
Contributor

Choose a reason for hiding this comment

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

Why is the json API depending on the Iteratee module?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

good question ;)
This is due to your Enumeratees which are in JSON object, Julien :D

I wondered if I should move them but it would have broken the API.
So first step was to create module like that and see if we keep this dep on
iteratees which is a bit superficial.
But anyway, this doesn't bring unwanted deps to people, this is pure
scala...

On Fri, Feb 22, 2013 at 12:04 PM, Julien Richard-Foy <
notifications@github.com> wrote:

In framework/project/Build.scala:

  •    )
    
  • ).settings(com.typesafe.sbtscalariform.ScalariformPlugin.defaultScalariformSettings: _*)
  • lazy val JsonProject = Project(
  •    "Play-Json",
    
  •    file("src/play-json"),
    
  •    settings = buildSettingsWithMIMA ++ Seq(
    
  •        previousArtifact := Some("play" % {"play-json_"+previousScalaVersion} % previousVersion),
    
  •        libraryDependencies := jsonDependencies,
    
  •        publishTo := Some(playRepository),
    
  •        scalacOptions ++= Seq("-encoding", "UTF-8", "-Xlint","-deprecation", "-unchecked", "-feature"),
    
  •        publishArtifact in packageDoc := buildWithDoc,
    
  •        publishArtifact in (Compile, packageSrc) := true
    
  •    )
    
  • ).settings(com.typesafe.sbtscalariform.ScalariformPlugin.defaultScalariformSettings: _*)
  • .dependsOn(IterateesProject, FunctionalProject, DataCommonsProject)

Why is the json API depending on the Iteratee module?


Reply to this email directly or view it on GitHubhttps://github.com//pull/754/files#r3114414.

Copy link
Contributor

Choose a reason for hiding this comment

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

Maybe we should put these Enumeratees in the play artifact?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

On Fri, Feb 22, 2013 at 12:12 PM, Julien Richard-Foy <
notifications@github.com> wrote:

In framework/project/Build.scala:

  •    )
    
  • ).settings(com.typesafe.sbtscalariform.ScalariformPlugin.defaultScalariformSettings: _*)
  • lazy val JsonProject = Project(
  •    "Play-Json",
    
  •    file("src/play-json"),
    
  •    settings = buildSettingsWithMIMA ++ Seq(
    
  •        previousArtifact := Some("play" % {"play-json_"+previousScalaVersion} % previousVersion),
    
  •        libraryDependencies := jsonDependencies,
    
  •        publishTo := Some(playRepository),
    
  •        scalacOptions ++= Seq("-encoding", "UTF-8", "-Xlint","-deprecation", "-unchecked", "-feature"),
    
  •        publishArtifact in packageDoc := buildWithDoc,
    
  •        publishArtifact in (Compile, packageSrc) := true
    
  •    )
    
  • ).settings(com.typesafe.sbtscalariform.ScalariformPlugin.defaultScalariformSettings: _*)
  • .dependsOn(IterateesProject, FunctionalProject, DataCommonsProject)

Maybe we should put these Enumeratees in the play artifact?

That was my first idea but didn't want to break it at first...
But this seems logical!


Reply to this email directly or view it on GitHubhttps://github.com//pull/754/files#r3114481.

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.

6 participants