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

dbeaver/dbeaver#35080 Add support for specifying additional JVM arguments #1

Open
wants to merge 8 commits into
base: master
Choose a base branch
from

Conversation

ShadelessFox
Copy link
Member

No description provided.

@n5a5
Copy link

n5a5 commented Oct 7, 2024

It looks good otherwise, so it's mostly some nitpicking

@ShadelessFox ShadelessFox requested a review from n5a5 October 7, 2024 12:47
Copy link

@n5a5 n5a5 left a comment

Choose a reason for hiding this comment

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

LGTM

@ShadelessFox ShadelessFox force-pushed the dbeaver/dbeaver#35080-additional-vmargs-locations branch from 18d706f to 97d186f Compare October 16, 2024 14:36
@ShadelessFox ShadelessFox force-pushed the dbeaver/dbeaver#35080-additional-vmargs-locations branch from 67033d4 to 6492f95 Compare October 16, 2024 16:56
@Matvey16
Copy link
Member

Verified

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
3 participants