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.

Sign up for GitHub

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

Jump to bottom

Load JSON parent first for logging #7521

Closed
wants to merge 1 commit into from
Closed

Load JSON parent first for logging #7521

wants to merge 1 commit into from

Conversation

stuartwdouglas
Copy link
Member

Fixes #7469

@boring-cyborg boring-cyborg bot added area/devtools Issues/PR related to maven, gradle, platform and cli tooling/plugins area/maven labels Mar 3, 2020
@jaikiran
Copy link
Member

jaikiran commented Mar 3, 2020

Hello @stuartwdouglas,

I had a test which was reproducing this issue. I have locally tested this with the same DevMojoIT testcase with your commit and it works fine. If needed, the test change is here https://github.com/jaikiran/quarkus/commit/2f3c1df9328b95d1c29a93130d9710b907709182 and can be included in this PR to prevent any future regressions.

@stuartwdouglas
Copy link
Member Author

replaced by #7523

@gsmet gsmet added the triage/invalid This doesn't seem right label Mar 5, 2020
@jaikiran jaikiran mentioned this pull request Sep 17, 2020
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
area/devtools Issues/PR related to maven, gradle, platform and cli tooling/plugins area/maven triage/invalid This doesn't seem right
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

quarkus-logging-json not working 1.3.0
3 participants
@stuartwdouglas @jaikiran @gsmet