-
Notifications
You must be signed in to change notification settings - Fork 90
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: replace [kK]ey[cC]loac?k with [kK]eycloak #519
fix: replace [kK]ey[cC]loac?k with [kK]eycloak #519
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good catch! I will do that 😉 |
Can you manage this before merging? ;) |
I will fix the typos after my vacation ;-) Just let's keep this PR open until I fixed them as well. |
a71153e
44f8e85
to
a71153e
Compare
I have left the following as it is because these occurrences refer to a script in another repo which I don't want to touch at the moment:
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Perfect.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Wanted to merge PR, but it has merge conflicts. Please resolve merge conflicts @florianrusch-zf |
@florianrusch, @carslen I could take over this PR, if needed |
i have already merged the changes locally and will push them later 😉
…________________________________
From: Stephan Bauer ***@***.***>
Sent: Wednesday, January 24, 2024 4:49:33 PM
To: eclipse-tractusx/eclipse-tractusx.github.io ***@***.***>
Cc: Florian Rusch ***@***.***>; Mention ***@***.***>
Subject: Re: [eclipse-tractusx/eclipse-tractusx.github.io] fix: replace [kK]ey[cC]loac?k with [kK]eycloak (PR #519)
Achtung: Diese E-Mail stammt von einem externen Absender. Bitte vermeide es, Anhänge oder externe Links zu öffnen
@florianrusch<https://github.com/florianrusch>, @carslen<https://github.com/carslen> I could take over this PR, if needed
—
Reply to this email directly, view it on GitHub<#519 (comment)>, or unsubscribe<https://github.com/notifications/unsubscribe-auth/AXGJYNX4USO32EOXYIWIWETYQEUQ3AVCNFSM6AAAAABACKFWR6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSMBYGQYDKMZXGA>.
You are receiving this because you were mentioned.Message ID: ***@***.***>
|
a71153e
to
7b7db15
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you very much @florianrusch for this effort
Description
There are 92 occurrences of the term "KeyCloak" in the repository. However, the correct spelling is "Keycloak" (source: https://www.keycloak.org/).
Fixes #518
Pre-review checks
Please ensure to do as many of the following checks as possible, before asking for committer review: