Skip to content
This repository was archived by the owner on Jun 23, 2020. It is now read-only.

Avoid deprecation/feature warnings #9

Merged
merged 5 commits into from
Sep 16, 2014
Merged

Conversation

sumito3478
Copy link
Contributor

This pull request updates scalaVersion to 2.11.2, sbt.version to 0.13.5, adds -deprecation and -feature to scalacOptions and fixes source code to avoid deprecation/feature warnings.

Since #8 is already working on the usages of deprecated CompilationUnit, this pull request does not fix them.

@retronym
Copy link
Member

This needs a (hopefully trivial) rebase as I've just merged #8.

This commit adds -deprecation and -feature to scalacOptions and fix source code to avoid deprecation/feature warnings.
Since scala#8 is already working on the usages of deprecated CompilationUnit, this commit does not fix them.
Testing with 2.11.0-SNAPSHOT results "UNRESOLVED DEPENDENCIES".
@sumito3478
Copy link
Contributor Author

@retronym Done, plus fixing the usages of deprecated CompilationUnit#warning (which is not fixed by #8), and updating sbt to 0.13.6.

Since testing with 2.11.0-SNAPSHOT results "UNRESOLVED DEPENDENCIES" (I don't know when it started to fail - downgrading to sbt 0.13.5 doesn't help), I updated the scala version to 2.11.0 in .travis.yaml.

@retronym
Copy link
Member

2.11.0-SNAPSHOT was expunged from the Sonatype OSS snapshots repository. 2.11.0 is okay. An alternative would by 2.11.3-SNAPSHOT, but that will eventually run into the same problem.

@retronym
Copy link
Member

LGTM, thanks for your attention to detail.

retronym added a commit that referenced this pull request Sep 16, 2014
Avoid deprecation/feature warnings
@retronym retronym merged commit a3a5ad1 into scala:master Sep 16, 2014
@retronym
Copy link
Member

One thing I forgot to ask is for you to review and agree to our CLA: http://typesafe.com/contribute/cla/scala

@sumito3478
Copy link
Contributor Author

@retronym Done.

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