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

Fix our crossScalaVersions since the last change to travis.yml #2771

Merged
merged 1 commit into from
Apr 3, 2019

Conversation

kailuowang
Copy link
Contributor

The sbt-travisci plugin was no longer able to parse travis.yml since we move away from build matrix. This, combined with other changes made to the build, broke sbt prePR as well snapshot release

@@ -743,7 +765,7 @@ addCommandAlias("validateKernelJS", "kernelLawsJS/test")
addCommandAlias("validateFreeJS", "freeJS/test") //separated due to memory constraint on travis
addCommandAlias("validate", ";clean;validateJS;validateKernelJS;validateFreeJS;validateJVM")

addCommandAlias("prePR", ";fmt;++2.11.12;validateBC")
addCommandAlias("prePR", ";fmt;++2.11.12 mimaReportBinaryIssues")
Copy link
Contributor Author

Choose a reason for hiding this comment

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

unfortunately, sbt doesn't seem to work when trying to fix to a scala version when running a command alias.
I filed an issue sbt/sbt#4599
by this change we no longer run binCompatTest/test, but I think it's rare that it's accidentally broken before PR.

@codecov-io
Copy link

Codecov Report

Merging #2771 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #2771   +/-   ##
=======================================
  Coverage   95.34%   95.34%           
=======================================
  Files         363      363           
  Lines        6785     6785           
  Branches      283      283           
=======================================
  Hits         6469     6469           
  Misses        316      316

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 1205af2...cc315fa. Read the comment docs.

Copy link
Member

@LukaJCB LukaJCB left a comment

Choose a reason for hiding this comment

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

Thanks for addressing this!

@kailuowang kailuowang merged commit abb97eb into typelevel:master Apr 3, 2019
@kailuowang kailuowang added this to the 2.0.0-M1 milestone Apr 18, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants