Skip to content

Introduce :launchpad/jvm-opts #14

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

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

arichiardi
Copy link
Contributor

@arichiardi arichiardi commented May 27, 2025

Make sure we populate the internal :java-args from the above-mentioned key, therefore allowing deps.local.edn to contain things that differ from deps.edn.

Note that there is naming discrepancy as this patch sticks with the official key - :jvm-opts rather than the internal name.

The behavior of appending to existing :jvm-opts, in order, has been preserved as per the tools.deps built-in.

Make sure we populate the internal :java-args from the above-mentioned
key, therefore allowing `deps.local.edn` to contain things that differ
from `deps.edn`.

Note that there is naming discrepancy as this patch sticks with the
official key - `:jvm-opts` rather than the internal name.

The behavior of appending to existing `:jvm-opts`, in order, has been
preserved as per the `tools.deps` built-in.
@arichiardi arichiardi changed the title Introduce :launchpad/jdk-opts Introduce :launchpad/jvm-opts May 27, 2025
@arichiardi arichiardi marked this pull request as ready for review May 27, 2025 19:53
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.

1 participant