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

Fix bug #2379

Merged
merged 3 commits into from
Sep 12, 2022
Merged

Fix bug #2379

merged 3 commits into from
Sep 12, 2022

Conversation

j-baker
Copy link
Contributor

@j-baker j-baker commented Sep 12, 2022

Before this PR

#2376 introduced LazyJdks package private interface that was not visible and triggered the following exception:

failed to access class com.palantir.baseline.plugins.javaversions.BaselineJavaVersionsExtension$LazyJdks from class com.palantir.gradle.jdks.JdksPlugin (com.palantir.baseline.plugins.javaversions.BaselineJavaVersionsExtension$LazyJdks and com.palantir.gradle.jdks.JdksPlugin are in unnamed module of loader org.gradle.internal.classloader.VisitableURLClassLoader @57e989a9)

After this PR

==COMMIT_MSG==
An interface was accidentally made package-private.
==COMMIT_MSG==

Possible downsides?

@changelog-app
Copy link

changelog-app bot commented Sep 12, 2022

Generate changelog in changelog/@unreleased

Type

  • Feature
  • Improvement
  • Fix
  • Break
  • Deprecation
  • Manual task
  • Migration

Description

Fix interface visibility bug

Check the box to generate changelog(s)

  • Generate changelog entry

@bulldozer-bot bulldozer-bot bot merged commit 383a79e into develop Sep 12, 2022
@bulldozer-bot bulldozer-bot bot deleted the jbaker/java_versions_ergonomics branch September 12, 2022 15:07
@svc-autorelease
Copy link
Collaborator

Released 4.163.0

This was referenced Sep 12, 2022
bulldozer-bot bot pushed a commit to palantir/witchcraft-api that referenced this pull request Sep 12, 2022
###### _excavator_ is a bot for automating changes across repositories.

Changes produced by the roomba/latest-baseline-oss check.

# Release Notes
## 4.162.0
| Type | Description | Link |
| ---- | ----------- | ---- |
| Feature | java versions preview enhancements<br><br>+ support setting preview on a project-by-project basis.<br>+ Fixes #2340<br>+ fails more elegantly if javaVersions is set on not-the-root. | palantir/gradle-baseline#2376 |


## 4.163.0
| Type | Description | Link |
| ---- | ----------- | ---- |
| Fix | Fix interface visibility bug | palantir/gradle-baseline#2379 |



To enable or disable this check, please contact the maintainers of Excavator.
This was referenced Sep 12, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants