8000 16188 added quotes to logging documentation to also work under Windows by sschu · Pull Request #16298 · keycloak/keycloak · GitHub
[go: up one dir, main page]

Skip to content

16188 added quotes to logging documentation to also work under Windows - #16298

Merged
vmuzikar merged 1 commit into
keycloak:mainfrom
bosch-io:16188_add_quotes_to_logging_documentation
Mar 1, 2023
Merged

vmuzikar merged 1 commit into
keycloak:mainfrom
bosch-io:16188_add_quotes_to_logging_documentation

Conversation

@sschu
@sschu sschu commented Jan 6, 2023
Copy link
Copy Markdown
Contributor

Fixes #16188

@sschu sschu changed the title 16188 added quotes do logging documentation to also work under Windows 16188 added quotes to logging documentation to also work under Windows Jan 9, 2023
@ahus1 ahus1 self-assigned this Feb 27, 2023
@ahus1 ahus1 added the area/docs Documentation — server guides, AsciiDoc, JavaDoc, migration/upgrade guides label Feb 27, 2023
@ahus1
ahus1 commented Feb 28, 2023
Copy link
Copy Markdown
Member

@sschu - could you please rebase this PR?

At the time you created this PR, check "Status Check - Keycloak JavaScript CI" didn't complete as it was misconfigured in the main branch at that time. As it is technically required, I hope that all required checks pass once you rebase this PR. Thanks!

@sschu
sschu force-pushed the 16188_add_quotes_to_logging_documentation branch from 0962e11 to e6109d5 Compare February 28, 2023 09:45
@sschu
sschu requested a review from a team as a code owner February 28, 2023 09:45
@sschu
sschu commented Feb 28, 2023
Copy link
Copy Markdown
Contributor Author

@ahus1 Sure, done.

@vmuzikar vmuzikar left a comment
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

@sschu Thanks, LGTM.

I believe the same issue also concerns the features guide. Feel free to open a follow-up issue, or update this PR. Either way works for me.

@ahus1
ahus1 commented Mar 1, 2023
Copy link
Copy Markdown
Member

@vmuzikar - we're about to move some docs, so it would be great if you could merge it rather now than later, otherwise this PR will be in conflict.

@vmuzikar vmuzikar mentioned this pull request Mar 1, 2023
2 tasks done
@vmuzikar vmuzikar left a comment
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

@ahus1 Works for me. Created #17376 for the Features guide.

@vmuzikar
vmuzikar merged commit edcbf73 into keycloak:main Mar 1, 2023
@jsorah jsorah mentioned this pull request May 16, 2023
2 tasks done
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/docs Documentation — server guides, AsciiDoc, JavaDoc, migration/upgrade guides

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Keycloak Quarkus Distro is not able to process the log multiple log handler

3 participants

0