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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

Gradle 8.2: work around fix for release publishing #3053

Merged
merged 2 commits into from Jul 5, 2023

Conversation

TWiStErRob
Copy link
Contributor

Missed a spot in :test warnings and workaround for shipkit/shipkit-changelog#103 triggered by https://github.com/mockito/mockito/actions/runs/5457593285/jobs/9931914322

See #3051 (comment)

Testing

I was able to get past the problem shown in the CI by setting env vars and applying the workaround. The failure is now at real publishing, so it should be ok 馃:

$ set NEXUS_TOKEN_USER=bad
$ set NEXUS_TOKEN_PWD=bad
$ gradlew githubRelease publishToSonatype closeAndReleaseStagingRepository releaseSummary

> Task :bom:publishJavaLibraryPublicationToSonatypeRepository FAILED
> Task :junit-jupiter:publishJavaLibraryPublicationToSonatypeRepository FAILED
> Task :proxy:publishJavaLibraryPublicationToSonatypeRepository FAILED
> Task :subclass:publishJavaLibraryPublicationToSonatypeRepository FAILED
> Task :errorprone:publishJavaLibraryPublicationToSonatypeRepository FAILED
> Task :android:publishJavaLibraryPublicationToSonatypeRepository FAILED
> Task :publishJavaLibraryPublicationToSonatypeRepository FAILED

all failures: Received status code 401 from server: Unauthorized

Checklist

  • Read the contributing guide
  • PR should be motivated, i.e. what does it fix, why, and if relevant how
  • If possible / relevant include an example in the description, that could help all readers
    including project members to get a better picture of the change
  • Avoid other runtime dependencies
  • Meaningful commit history ; intention is important please rebase your commit history so that each
    commit is meaningful and help the people that will explore a change in 2 years
  • The pull request follows coding style
  • Mention Fixes #<issue number> in the description if relevant
  • At least one commit should mention Fixes #<issue number> if relevant

> Task :extTest:test
The automatic loading of test framework implementation dependencies has been deprecated. This is scheduled to be removed in Gradle 9.0. Declare the desired test framework directly on the test suite or explicitly declare the test framework implementation dependencies on the test's runtime classpath. Consult the upgrading guide for further information: https://docs.gradle.org/8.2/userguide/upgrading_version_8.html#test_framework_implementation_dependencies
Copy link
Contributor

@TimvdLippe TimvdLippe left a comment

Choose a reason for hiding this comment

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

Thanks!

@TimvdLippe TimvdLippe merged commit c1eec88 into mockito:main Jul 5, 2023
13 checks passed
@TWiStErRob TWiStErRob deleted the gradle8-2 branch July 5, 2023 10:03
TWiStErRob added a commit to TWiStErRob/mockito that referenced this pull request Jan 27, 2024
TimvdLippe pushed a commit that referenced this pull request Jan 27, 2024
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

Successfully merging this pull request may close these issues.

None yet

2 participants