Skip to content

Another round of renaming / preparing for open-source release #5

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

Merged
merged 7 commits into from
Mar 24, 2014

Conversation

retronym
Copy link
Member

Details in the commits.

@retronym
Copy link
Member Author

Review by @gkossakowski @adriaanm

@adriaanm Can you please create a team for this repo and add the Akka guys as collaborators? Alternatively, we can merge this PR and make the repo public.

   "org.scala-lang.modules" %% "scala-compat-jdk8"

   import scala.compat.jdk8._

Subsequent commits will address the under-loved `JFunc`.
In accordance with the move to the Scala github organization
and the plan to release this under "org.scala-lang.modules".
So as to avoid the inconsistent abbreviation with `JFunctionN`.
  - add the scala-modules SBT plugin
  - configure OSGi package export
  - Check that SBT is running on Java8, rather than waiting for
    cryptic build errors.
  - convert `LambdaTest` to a JUnit test.
@retronym
Copy link
Member Author

I'm now undecided between scala-java8-compat and scala-compat-java8 as the artifact name. I think I prefer the former. Thoughts?

@retronym
Copy link
Member Author

In either case, we need to rename the repo when we merge this PR.

@gkossakowski
Copy link

I prefer the former as well. Reading aloud: "scala-java8-compat" and "scala-compat-java8" helps to make one's mind. The former is easier to talk about.

@gkossakowski
Copy link

Yes, github supports (only once per repo) renames though.

@gkossakowski
Copy link

The argument in favor of scala-compat-java8 is that we use scala.compat.java8 package name. However, I think in this case it's fine that the name of the module and the package name deviate.

@retronym
Copy link
Member Author

I think I've convinced myself that having it around backwards actually correct because package names are reversed anyway; people say typesafe.com, not com.typesafe.

@gkossakowski
Copy link

I think I've convinced myself that having it around backwards

"It" refers to module name here?

@retronym
Copy link
Member Author

Yep, "having the segments of module name and package names reversed".

@gkossakowski
Copy link

The overall diff LGTM. Let's squash some of the commits that amend other commits in this PR and we are ready to go. If it's too much work to squash commits then feel free to merge as is.

@retronym
Copy link
Member Author

This one will be fiddly to rebase (references to names in docs, build etc are hard to catch to be sure each step is consistent.) So I think I'll take the free pass on this one.

retronym added a commit that referenced this pull request Mar 24, 2014
Another round of renaming / preparing for open-source release
@retronym retronym merged commit 5878e9f into scala:master Mar 24, 2014
scalaVersion := "2.10.3"
import com.typesafe.tools.mima.plugin.{MimaPlugin, MimaKeys}

scalaModuleSettings
Copy link
Contributor

Choose a reason for hiding this comment

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

yay!

@retronym retronym added this to the 0.1.0 milestone Mar 26, 2014
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.

3 participants