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

Don't use system dependent charset #24025

Merged
merged 1 commit into from
Mar 1, 2022
Merged

Conversation

stuartwdouglas
Copy link
Member

@stuartwdouglas stuartwdouglas commented Mar 1, 2022

Fixes #23991
Fixes #23914

Copy link
Member

@phillip-kruger phillip-kruger left a comment

Choose a reason for hiding this comment

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

Nice ! Thanks @stuartwdouglas

@quarkus-bot
Copy link

quarkus-bot bot commented Mar 1, 2022

This workflow status is outdated as a new workflow run has been triggered.

Failing Jobs - Building 6e4b68d

Status Name Step Failures Logs Raw logs
Initial JDK 11 Build Build Failures Logs Raw logs

Failures

⚙️ Initial JDK 11 Build #

- Failing: extensions/vertx-http/runtime 
! Skipped: core/test-extension/deployment core/test-extension/runtime devtools/bom-descriptor-json and 438 more

📦 extensions/vertx-http/runtime

Failed to execute goal net.revelc.code:impsort-maven-plugin:1.6.2:check (check-imports) on project quarkus-vertx-http: Imports are not sorted in /home/runner/work/quarkus/quarkus/extensions/vertx-http/runtime/src/main/java/io/quarkus/vertx/http/runtime/devmode/FileSystemStaticHandler.java

@geoand geoand added triage/waiting-for-ci Ready to merge when CI successfully finishes triage/backport? labels Mar 1, 2022
@gsmet gsmet merged commit 0480ac4 into quarkusio:main Mar 1, 2022
@quarkus-bot quarkus-bot bot added this to the 2.8 - main milestone Mar 1, 2022
@quarkus-bot quarkus-bot bot added kind/bugfix and removed triage/waiting-for-ci Ready to merge when CI successfully finishes labels Mar 1, 2022
@gsmet
Copy link
Member

gsmet commented Mar 1, 2022

I added #23914 to the fixed issues as I think it will solve this other issue as well.

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.

Swagger UI don't show in dev mode from 2.7.0.Final graphql-ui issue show  on each line and near the cursor
5 participants