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

Use scala versions define in .travis.yml #1490

Merged
merged 1 commit into from
Dec 20, 2016
Merged

Use scala versions define in .travis.yml #1490

merged 1 commit into from
Dec 20, 2016

Conversation

dwijnand
Copy link
Contributor

Using sbt-travisci crossScalaVersions and scalaVersion are defined in terms of the scala versions defined in .travis.yml. Specifically, for scalaVersion the last scala version is used (therefore either re-order them or override it in build.sbt if it's not the default you want).

Also, define organization in ThisBuild once instead of defining it in buildSettings and appending it to every single project (in project scope).

Finally, remove use of %% inside addSbtPlugin, it's overridden anyways by addSbtPlugin.

@codecov-io
Copy link

codecov-io commented Dec 10, 2016

Current coverage is 91.94% (diff: 100%)

Merging #1490 into master will not change coverage

@@             master      #1490   diff @@
==========================================
  Files           244        244          
  Lines          3623       3623          
  Methods        3502       3502          
  Messages          0          0          
  Branches        121        121          
==========================================
  Hits           3331       3331          
  Misses          292        292          
  Partials          0          0          

Powered by Codecov. Last update f4d47af...0f427f7

@kailuowang
Copy link
Contributor

👍

Using sbt-travisci 'crossScalaVersions' and 'scalaVersion' are defined
in terms of the scala versions defined in .travis.yml. Specifically, for
'scalaVersion' the last scala version is used (therefore either re-order
them or override it in build.sbt if it's not the default you want).

Also, define 'organization in ThisBuild' once instead of defining it in
'buildSettings' and appending it to every single project (in project
scope).

Finally, remove use of '%%' inside 'addSbtPlugin', it's overridden
anyways by 'addSbtPlugin'.
@dwijnand dwijnand mentioned this pull request Dec 20, 2016
@dwijnand
Copy link
Contributor Author

The irony that this conflicts with #1499, which is a 2 commit PR, first updating build.sbt and then .travis.yml :-) (and then mentioning sbt-travisci)

@adelbertc
Copy link
Contributor

Hurrah! Thanks @dwijnand ! And thanks to @djspiewak for making me aware of this plugin

@adelbertc adelbertc merged commit c9521e1 into typelevel:master Dec 20, 2016
@dwijnand
Copy link
Contributor Author

Excellent. Thanks @adelbertc.

@dwijnand dwijnand deleted the sbt-travisci branch December 20, 2016 20:31
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.

4 participants