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

release scalacheck_sjs0.6_2.13.0-M1 #330

Closed
xuwei-k opened this issue Apr 29, 2017 · 16 comments
Closed

release scalacheck_sjs0.6_2.13.0-M1 #330

xuwei-k opened this issue Apr 29, 2017 · 16 comments

Comments

@xuwei-k
Copy link
Contributor

xuwei-k commented Apr 29, 2017

@rickynils

@xuwei-k
Copy link
Contributor Author

xuwei-k commented May 25, 2017

ping

@djspiewak
Copy link
Member

This is actually holding up a good chunk of the ecosystem, since most projects cannot release their SJS 2.13.0-M1 cross-builds without ScalaCheck and ScalaTest.

@rickynils
Copy link
Contributor

The doc task seems to fail due to changes in ScalaJS (bumped to 0.6.17 to get the 2.13.0-M1 release):

> doc
[info] Main Scala API documentation to /home/rickard/src/scalacheck/js/target/scala-2.13.0-M1/api...
[error] /home/rickard/src/scalacheck/src/main/scala/org/scalacheck/Prop.scala:25: @JSExportDescendentClasses is deprecated and will be removed in 1.0.0. For use cases where you want to simulate "reflective" instantiation, use @EnableReflectiveInstantion and scala.scalajs.reflect.Reflect.lookupInstantiatableClass instead.
[error]   (you can suppress this warning in 0.6.x by passing the option `-P:scalajs:suppressExportDeprecations` to scalac)
[error] @Platform.JSExportDescendentClasses
[error]  ^
[error] /home/rickard/src/scalacheck/src/main/scala/org/scalacheck/Prop.scala:26: @JSExportDescendentObjects is deprecated and will be removed in 1.0.0. For use cases where you want to simulate "reflective" loading, use @EnableReflectiveInstantion and scala.scalajs.reflect.Reflect.lookupLoadableModuleClass instead.
[error]   (you can suppress this warning in 0.6.x by passing the option `-P:scalajs:suppressExportDeprecations` to scalac)
[error] @Platform.JSExportDescendentObjects
[error]  ^
[error] /home/rickard/src/scalacheck/src/main/scala/org/scalacheck/Properties.scala:29: @JSExportDescendentClasses is deprecated and will be removed in 1.0.0. For use cases where you want to simulate "reflective" instantiation, use @EnableReflectiveInstantion and scala.scalajs.reflect.Reflect.lookupInstantiatableClass instead.
[error]   (you can suppress this warning in 0.6.x by passing the option `-P:scalajs:suppressExportDeprecations` to scalac)
[error] @Platform.JSExportDescendentClasses
[error]  ^
[error] /home/rickard/src/scalacheck/src/main/scala/org/scalacheck/Properties.scala:30: @JSExportDescendentObjects is deprecated and will be removed in 1.0.0. For use cases where you want to simulate "reflective" loading, use @EnableReflectiveInstantion and scala.scalajs.reflect.Reflect.lookupLoadableModuleClass instead.
[error]   (you can suppress this warning in 0.6.x by passing the option `-P:scalajs:suppressExportDeprecations` to scalac)
[error] @Platform.JSExportDescendentObjects
[error]  ^
[info] No documentation generated with unsuccessful compiler run
[error] four errors found
[error] (js/compile:doc) Scaladoc generation failed
[error] Total time: 6 s, completed Jun 8, 2017 4:37:47 PM

@djspiewak
Copy link
Member

Ping @sjrd.

@xuwei-k
Copy link
Contributor Author

xuwei-k commented Jun 8, 2017

just remove -Xfatal-warnings ?

https://github.com/rickynils/scalacheck/blob/1.13.5/build.sbt#L51

@djspiewak
Copy link
Member

That's an option: https://github.com/djspiewak/scalacheck/tree/wip/2.13 It'll have to be addressed eventually though.

@SethTisue
Copy link
Member

http://www.scala-js.org/news/2017/03/21/announcing-scalajs-0.6.15/ explains the deprecation and how to migrate

@sjrd
Copy link
Contributor

sjrd commented Jun 8, 2017

As pointed by @SethTisue, the 0.6.15 release notes should explain everything you need to know to migrate. Feel free to ask further questions if it's not enough.

There's even a "Are you maintaining a testing framework?" section, just for you guys :P

@SethTisue
Copy link
Member

@rickynils is your wip on this on GitHub? I could try to finish it up.

@rickynils
Copy link
Contributor

@SethTisue No wip. I just bumbed the scalajs version in plugins.sbt. If PRs are provided for the 1.13.5_sonatype and 1.12.6_sonatype branches I can try to publish releases. But maybe we have to cut new releases if there are code changes.

Sorry for my unresponsiveness. It is mainly because I got a kid some weeks ago, and partly because I have never used ScalaJS.

@SethTisue
Copy link
Member

please continue to prioritize your human children over your digital ones :-)

the release notes say that scalacOptions += "-P:scalajs:suppressExportDeprecations" will continue to work for Scala.js 0.6.x, so that seems like an acceptable minimal fix for now. I will send PRs on the two branches you suggest

@SethTisue
Copy link
Member

trio of pull requests: #339, #340, #341

@SethTisue
Copy link
Member

hmm, actually it's not that hard to go ahead and make the necessary changes to not use the deprecated stuff at all. revising the PRs now

@SethTisue
Copy link
Member

hmm, well, probably better to make the minimal change on the release branches. I did the better version on master (#341); the other two are fine as-is I think.

@xuwei-k
Copy link
Contributor Author

xuwei-k commented Jun 25, 2017

@xuwei-k xuwei-k closed this as completed Jun 25, 2017
@rickynils
Copy link
Contributor

@SethTisue Thanks for the PRs! I've published releases for ScalaCheck 1.12.6 and 1.13.5

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

No branches or pull requests

5 participants