-
Notifications
You must be signed in to change notification settings - Fork 2.7k
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
Fixes #520 accept debugger by default in dev mode #523
Conversation
* | ||
* false: The JVM is not started in debug mode true: The JVM is started in debug mode and suspends until a debugger is | ||
* attached to 5005 client: The JVM is started in client mode, and attempts to connect to localhost:5005 {port}: The JVM is | ||
* started in debug mode and suspends until a debugger is attached to {port} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Maybe it would be better with a proper list? It's not very readable right now.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
+1 for an unordered list!
//to the runner jar | ||
// we also want to add the maven plugin jar to the class path | ||
// this allows us to just directly use classes, without messing around copying them | ||
// to the runner jar |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This PR is a good example of why we need a formatter.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I just realised I had accidentally left the eclipse formatter enabled after my experiments
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to me apart from the comment on the Javadoc that could be improved.
@stuartwdouglas nice table ;-) |
Co-authored-by: allcontributors[bot] <46447321+allcontributors[bot]@users.noreply.github.com>
No description provided.