-
Notifications
You must be signed in to change notification settings - Fork 59
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
Inconsistent release 1.6.17 #351
Comments
I noticed the release job failed yesterday, but on the bintray stage, didn't know that the maven release was corrupted... Thanks for reporting it! |
Hey guys! This is a little strange.
@bbonanno, what are the expected covariants of Scala we should be publishing? |
For whatever reason, the .pom is missing from the published artifact: https://dl.bintray.com/mockito/maven/org/mockito/mockito-scala_2.13/1.16.17/ Note that the .pom.md5 is there, so that's a little strange. |
The problem is that the CI build failed to publish all artifacts (details). My theory is that it's a bug in Gradle Bintray plugin: when one of the |
@bbonanno, |
Until #351 is fixed, let's avoid automated publications to Maven Central
@mockitoguy yeah, I noticed that after I merged, but in any case, it should have done a full release right? (I didn't care cause a full release out of that PR was not the end of the world). The full release is a cartesian product of all the modules that produce a Jar x the 3 Scala versions we support |
I just merged another PR, no comments, so let's see how the release goes |
went good :) the new release is consistent |
Nice, let's hope it doesn't happen again! |
Had to release again (another fix was made to the lib) so if that one's also successful I'll close this issue |
Hey @bbonanno, let's keep this open for a little bit because the root cause is not fixed. I reported a bug to Bintray plugin: bintray/gradle-bintray-plugin#340 |
@mockitoguy oh, fair enough |
@mockitoguy this thing got worse, the last 3 master builds all hung and got cancelled after being stuck in the release step for a while |
I will look into this. We need to migrate off bintray anyway. |
another failed release :( |
Hi @mockitoguy, sorry to bother but I noticed that even the PR builds are failing now on the Bintray upload, did something change around that? |
Will take a look!
On Thu, Jul 29, 2021 at 11:02 Bruno Bonanno ***@***.***> wrote:
Hi @mockitoguy <https://github.com/mockitoguy>, sorry to bother but I
noticed that even the PR builds are failing now on the Bintray upload, did
something change around that?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#351 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AAAGBJ5GFNWX75URTMSJEDTT2EKKXANCNFSM4WWQDBGQ>
.
--
Szczepan (Stefan) Faber
Creator of Mockito <http://mockito.org> | Maker of Open Source on GitHub
<https://github.com/mockitoguy>
Check out how to give great code reviews
<https://thenewstack.io/linkedin-code-review>. You will be glad you did!
|
Hi @mockitoguy, sorry to be annoying but now no release at all can be made, and PRs don't build anymore as the bintrayUpload always fails... seems to be related to credentials according to the error, but there isn't much I can do here, would you mind to have a look :) Cheers!!! |
I'll make a commitment and resolve it by the end of this week. Will this
work? Thanks for your patience! I've been traveling.
|
@mockitoguy sure, no worries, enjoy your trip :) |
Let's track this in #320 |
Scala steward informed me about a new release 1.6.17.
Looking at https://search.maven.org/search?q=mockito-scala it seems that something went wrong in the process, some packages have the new version some the old. Even mixing for different scala versions.
Not a real problem for me, but I thought you might to know (if you did not already noticed it)
The text was updated successfully, but these errors were encountered: