-
Notifications
You must be signed in to change notification settings - Fork 3k
Add ability to change args between restarts #18286
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
Add ability to change args between restarts #18286
Conversation
This can be used for more than just testing now.
4e42804
to
756465f
Compare
This workflow status is outdated as a new workflow run has been triggered. 🚫 This workflow run has been cancelled. Failing Jobs - Building 4e42804
|
756465f
to
412849a
Compare
This workflow status is outdated as a new workflow run has been triggered. |
412849a
to
ace5b32
Compare
This workflow status is outdated as a new workflow run has been triggered. |
ace5b32
to
cb3738e
Compare
This workflow status is outdated as a new workflow run has been triggered. |
cb3738e
to
1de1234
Compare
This workflow status is outdated as a new workflow run has been triggered. |
- Large stack traces would loose color - Large stack traces can loose some output - Use 'default forground' instead of 'reset all'
1de1234
to
3c411e4
Compare
This workflow status is outdated as a new workflow run has been triggered. |
Failing Jobs - Building 3c411e4
Full information is available in the Build summary check run. Test Failures⚙️ Maven Tests - JDK 11 #📦 integration-tests/maven✖ |
...ment/src/main/java/io/quarkus/vertx/http/deployment/devmode/console/DevConsoleProcessor.java
Show resolved
Hide resolved
Should this be backported? Asking because it actually fixes #18287 |
We can try, it may create conflicts though. |
Added the label, I'll have a look this afternoon. |
If we can't backport it then we should do another commit with the null checks from BasicConsole. |
Also add the ability to open a browser window for HTTP based apps