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

Add 2.13.0-M3 build #2173

Closed
wants to merge 1 commit into from
Closed

Add 2.13.0-M3 build #2173

wants to merge 1 commit into from

Conversation

LukaJCB
Copy link
Member

@LukaJCB LukaJCB commented Feb 20, 2018

Should fix #1648

@codecov-io
Copy link

codecov-io commented Feb 20, 2018

Codecov Report

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

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #2173   +/-   ##
=======================================
  Coverage   94.66%   94.66%           
=======================================
  Files         328      328           
  Lines        5548     5548           
  Branches      213      213           
=======================================
  Hits         5252     5252           
  Misses        296      296

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 5861210...6a634f7. Read the comment docs.

kailuowang
kailuowang previously approved these changes Feb 20, 2018
Copy link
Contributor

@kailuowang kailuowang left a comment

Choose a reason for hiding this comment

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

on build green.

@johnynek
Copy link
Contributor

+1 on green

@kailuowang
Copy link
Contributor

kailuowang commented Feb 22, 2018

blocked by scoverage/scalac-scoverage-plugin#215 now.

@kailuowang
Copy link
Contributor

might need to bump scalajs to 0.6.22 as well.

@johnynek
Copy link
Contributor

johnynek commented Feb 22, 2018 via email

@kailuowang
Copy link
Contributor

@johnynek I mistyped. sorry about the noise. anyway this is still blocked by scalac-scoverage-plugin

@SethTisue
Copy link
Member

SethTisue commented Feb 23, 2018

scalac-scoverage-plugin has now been published for 2.13.0-M3

and yes, you'll definitely need to bump Scala.js to 0.6.22

@LukaJCB LukaJCB force-pushed the 13-m3 branch 3 times, most recently from 252d909 to 5ae48f9 Compare February 23, 2018 09:27
@LukaJCB
Copy link
Member Author

LukaJCB commented Feb 23, 2018

We're still blocked by catalysts and export-hook

@milessabin
Copy link
Member

export-hook now published for 2.13.0-M3.

@ghost
Copy link

ghost commented Feb 23, 2018

We're still blocked by catalysts....

Looking now....

@ghost
Copy link

ghost commented Feb 25, 2018

cc'ing from gitter chat

BennyHill @BennyHill 17:58
So re 2.13.0-M3. Whilst updating catalysts, these are the current issues that I'm coming across (that a cats build would also run into directly if it were not to use catalysts):
discipline has been updated for 2.13.0-M3, but not released
tut still on 2.13.0-M1
I'm guessing microsites would need a new release after a tut release
So for catalysts, I can (try to) build these locally and carry on with any changes in the meantime
paging @tpolecat @larsrh re ^^

Rob Norris @tpolecat 18:07
The REPL interface changed in 2.13.0-M2 so I need to fork the tut source by version to support 2.13.

@ghost
Copy link

ghost commented Feb 25, 2018

Additionally, last year when I compiled for cats with 2.13.0-M3 there were a few code changes required for simulacrum, that were not too obvious. And for scalatest, a milestone release is required.

It might be easier if I just push an new PR in the next few days. Any objections if I did that? Obviously we can keep this open in the meantime

@tpolecat
Copy link
Member

I'll try to get tut working for 2.13-M2+ later this week. It's going to be terrible. I can feel it.

@SethTisue
Copy link
Member

@tpolecat
rsz_eeyore61_5881_5847

let us know if you get stuck, we'll help.

@SethTisue SethTisue mentioned this pull request Feb 26, 2018
@tpolecat
Copy link
Member

tut 0.6.3-M5 appears to work with 2.13.0-M3 … I'll do a final in a day or two if nothing comes up. So @BennyHill you may or may not want to wait.

@ghost
Copy link

ghost commented Feb 26, 2018

@tpolecat Awesome! Let's see how far I can get with this 👍

@ghost
Copy link

ghost commented Mar 14, 2018

The background work for this is effectively done (cats PR, discipline PR, new tut, new microsites), but the build is faiing due to, I beileve, a 2.13.0-M3 regression. See https://gist.github.com/BennyHill/6e4f698fe7f85309caeb8fffdf474fd0 and scalaz/scalaz#1629

@ghost
Copy link

ghost commented Mar 14, 2018

@ghost
Copy link

ghost commented Mar 14, 2018

@LukaJCB , @kailuowang

Suggestion:

  • rename this PR to "Updates in prep for 2.13.0-M3 build" or similar
  • remove - 2.13.0-M3 from .travis.yml
  • rebase on master
  • release

That way we get the newer versions of these libs/plugins, regardless of any other work we do for 2.13

However, catalysts changes for 2.13.0-M3 are done typelevel/catalysts#16 so tomorrow/Friday I will release and also PR the newer changes specific for cats

The changes are actually minimal, and the build will fail because of scala issues. This way, it's easier for others to see the changes and, if they are not fixed for cats 1.1, at least we get your upgrades from above in,

@djspiewak
Copy link
Member

@BennyHill @kailuowang @SethTisue Any idea on the current status of this? Is it actually a regression in scalac? And if so, will it get fixed in M4?

@kailuowang
Copy link
Contributor

@djspiewak @BennyHill did a lot of work on this one, which is merged in #2203. The build fails due to scala/bug#10706, which seems to be fixed and scheduled to release in M4.

@ghost
Copy link

ghost commented May 1, 2018

Should this PR be closed now #2203 is merged?

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.

publish for 2.13.0-M3
9 participants