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

Javadoc tasks supporty BaselineModuleJvmArgs #2137

Merged
merged 3 commits into from
Mar 28, 2022

Conversation

carterkozak
Copy link
Contributor

This unblocks the error-prone 2.11 upgrade.

==COMMIT_MSG==
Javadoc tasks supporty BaselineModuleJvmArgs
==COMMIT_MSG==

This unblocks the error-prone 2.11 upgrade.
@carterkozak carterkozak requested a review from fawind March 28, 2022 16:39
@changelog-app
Copy link

changelog-app bot commented Mar 28, 2022

Generate changelog in changelog/@unreleased

Type

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

Description

Javadoc tasks supporty BaselineModuleJvmArgs

Check the box to generate changelog(s)

  • Generate changelog entry

if (!exportValues.isEmpty()) {
coreOptions
// options are automatically prefixed with '-' internally
.addMultilineStringsOption("-add-exports")
Copy link
Contributor

Choose a reason for hiding this comment

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

Was wondering why we can't use collectClasspathArgs here which does a very similar colelction of the export and open args. Is it because of the -- vs - distinction?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Sort of, add-exports only occurs once, and the values are collected on the right-hand-side.

@bulldozer-bot bulldozer-bot bot merged commit f804a6c into develop Mar 28, 2022
@bulldozer-bot bulldozer-bot bot deleted the ckozak/javadoc_opens_exports branch March 28, 2022 17:10
@svc-autorelease
Copy link
Collaborator

Released 4.81.0

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

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

# Release Notes
## 4.80.0
| Type | Description | Link |
| ---- | ----------- | ---- |
| Improvement | Compilation supports add-exports/add-opens with toolchains | palantir/gradle-baseline#2136 |


## 4.81.0
| Type | Description | Link |
| ---- | ----------- | ---- |
| Improvement | Javadoc tasks supporty BaselineModuleJvmArgs | palantir/gradle-baseline#2137 |


## 4.82.0
_Automated release, no documented user facing changes_


To enable or disable this check, please contact the maintainers of Excavator.
This was referenced Mar 28, 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.

4 participants