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

Cut a release for Guice 4.0.0 final #45

Closed
mushtaq opened this issue May 6, 2015 · 12 comments
Closed

Cut a release for Guice 4.0.0 final #45

mushtaq opened this issue May 6, 2015 · 12 comments
Assignees
Milestone

Comments

@mushtaq
Copy link

mushtaq commented May 6, 2015

No description provided.

@tsuckow tsuckow self-assigned this May 9, 2015
@tsuckow
Copy link
Member

tsuckow commented May 9, 2015

I can't believe they finally did it. I should look at this Monday as I am out of town.

@tsuckow
Copy link
Member

tsuckow commented May 12, 2015

I've updated the snapshot version 4.0.0-SNAPSHOT. Give that a spin and if the wheels don't fall off let me know and I'll promote it to 4.0.0

@mushtaq
Copy link
Author

mushtaq commented May 12, 2015

I tried using snapshot but sbt could not resolve it. Do I have to add a new
resolver?

Thanks.

On Tue, May 12, 2015 at 9:30 AM, Thomas Suckow notifications@github.com
wrote:

I've updated the snapshot version 4.0.0-SNAPSHOT. Give that a spin and if
the wheels don't fall off let me know and I'll promote it to 4.0.0


Reply to this email directly or view it on GitHub
#45 (comment)
.

@tsuckow
Copy link
Member

tsuckow commented May 12, 2015

Well this is embarrassing. All this time I thought I was pushing SNAPSHOT releases I had a typo.

It should "just work" now. But if not it is at https://oss.sonatype.org/content/repositories/snapshots/

Or if you wanted to download the jar directly for some reason oss.sonatype.org/.../scala-guice_2.11/4.0.0-SNAPSHOT/

@mushtaq
Copy link
Author

mushtaq commented May 12, 2015

Ok, after I added resolvers += Resolver.sonatypeRepo("snapshots"), all my
tests pass on local box. But I can not push it to CI as snapshots repo are
not accessible from CI. So all green from side.

Thanks.

On Tue, May 12, 2015 at 11:17 AM, Thomas Suckow notifications@github.com
wrote:

Well this is embarrassing. All this time I thought I was pushing SNAPSHOT
releases I had a typo.

It should "just work" now. But if not it is at
https://oss.sonatype.org/content/repositories/snapshots/

Or even more specifically
oss.sonatype.org/.../scala-guice_2.11/4.0.0-SNAPSHOT/
https://oss.sonatype.org/content/repositories/snapshots/net/codingwell/scala-guice_2.11/4.0.0-SNAPSHOT/


Reply to this email directly or view it on GitHub
#45 (comment)
.

@nlochschmidt
Copy link

Just tested the 4.0.0-SNAPSHOT in our project and everything works as expected. And as a bonus the Guava evicted messages (#40) will finally be fixed \o/.

@tsuckow
Copy link
Member

tsuckow commented May 20, 2015

Good to hear!

I am trying to get #44 squared away. I am swamped, I hope to make a new snapshot tonight changing the related dependency and then hopefully get the full release out before the long weekend.

@nbauernfeind
Copy link
Member

I was almost done with #42 well back in the day. Does it make sense to finish that off? An almost complete implementation is here: https://github.com/nbauernfeind/scala-guice/tree/TypeTag.

However, I think we would need to drop support for scala 2.10 since TypeTag's are not thread safe (see: http://docs.scala-lang.org/overviews/reflection/thread-safety.html). 2.11 has been release for only about a year (since 4/16/2014).

Or, if not dropping support, we could branch off of 2.10 but some changes would not be portable. Maybe we could have one last 2.10 release to celebrate Guice's 4.0?

@tsuckow
Copy link
Member

tsuckow commented May 20, 2015 via email

@tsuckow
Copy link
Member

tsuckow commented May 21, 2015

Alright. I have pushed a new SNAPSHOT that fixes #44. Please give it a spin and make sure that simple change hasn't broke the world. If no one protests I plan to release 4.0.0 Friday before I leave on holiday. Then we should see about merging #42 for 4.0.1

@tsuckow tsuckow added this to the 4.0.0 milestone May 21, 2015
@tsuckow
Copy link
Member

tsuckow commented May 26, 2015

Should be synced to maven central shortly

@tsuckow tsuckow closed this as completed May 26, 2015
@mushtaq
Copy link
Author

mushtaq commented May 27, 2015

Cool, thanks.

On Tue, May 26, 2015 at 7:36 AM, Thomas Suckow notifications@github.com
wrote:

Should be synced to maven central shortly


Reply to this email directly or view it on GitHub
#45 (comment)
.

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

4 participants