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

Fix user-facing typos in Java code #28971

Merged
merged 1 commit into from May 6, 2024

Conversation

DimitriPapadopoulos
Copy link
Contributor

This includes documentation and user-visible messages, but excludes log messages.

This is a follow-up to #28890. This is currently an incomplete draft, I want to make sure I have got "only user-facing content excluding log messages" right.

@ahus1
Copy link
Contributor

ahus1 commented May 6, 2024

@DimitriPapadopoulos - some of the tests fail as they test for error messages in the responses.
I'd prefer (but not require you) to split this PR into two parts: One that changes JavaDoc only, and another one that changes error messages which then would require also changes to the tests.

Copy link
Contributor

@thomasdarimont thomasdarimont left a comment

Choose a reason for hiding this comment

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

LGTM. Thanks for your contribution! Error messages in tests need to be aligned, of course.

@DimitriPapadopoulos
Copy link
Contributor Author

DimitriPapadopoulos commented May 6, 2024

I have modified this pull request to include only fixes in Javadoc.

Changes to error messages will follow in a distinct merge request (#29326).

Signed-off-by: Dimitri Papadopoulos <3234522+DimitriPapadopoulos@users.noreply.github.com>
@ahus1 ahus1 enabled auto-merge (squash) May 6, 2024 18:54
@ahus1 ahus1 self-assigned this May 6, 2024
@ahus1
Copy link
Contributor

ahus1 commented May 6, 2024

Thank you for this PR, @DimitriPapadopoulos. Once you start changing Java code, please open a GitHub issue first and then link it to the PR as outlined in the CONTRIBUTING.md guide.

@ahus1 ahus1 merged commit cd8e0fd into keycloak:main May 6, 2024
69 checks passed
@DimitriPapadopoulos DimitriPapadopoulos deleted the codespell_java branch May 6, 2024 19:10
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.

None yet

4 participants