jclouds-notifications mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Ignasi Barrera <notificati...@github.com>
Subject Re: [jclouds] Adds autovalue/gson builders tests, for sanity. (#699)
Date Tue, 03 Mar 2015 17:15:55 GMT
-1

While it only affects the tests, we are opening the door to anyone to use it by mistake, and
this must **not** be used until there is a fixed release.

Also, we **only** release the source code, and the tarballs we release must produce the same
articafts in each build, at any time. We can't guarantee that by adding a SNAPSHOT dependency.
It could also break the tests of an already published release!

I'm -1 to have this merged to master but also understand the value of having these tests to
allow us to move forward faster once it is released. I suggest to push a branch with this
change (say, auto-1.1) and to add a Jenkins build that verifes it, but SNAPSHOT dependencies
can't be merged into master.

---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds/pull/699#issuecomment-76989512
Mime
View raw message