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: replace [kK]ey[cC]loac?k with [kK]eycloak #519

Merged

Conversation

florianrusch-zf
Copy link
Contributor

@florianrusch-zf florianrusch-zf commented Dec 1, 2023

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:

⚠️ I didn't tick the last checkbox because there are a lot of files that have no copyright and licence header at all. I believe this should be added in a separate PR rather than this one.

tomaszbarwicki
tomaszbarwicki previously approved these changes Dec 5, 2023
Copy link
Contributor

@tomaszbarwicki tomaszbarwicki left a comment

Choose a reason for hiding this comment

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

LGTM

FaGru3n
FaGru3n previously approved these changes Dec 5, 2023
Copy link
Contributor

@FaGru3n FaGru3n left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Contributor

@drcgjung drcgjung left a comment

Choose a reason for hiding this comment

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

very neat initiative. You could, however, also replace occurences of "[kK]ey[cC]loack" in 6 files.

image

@florianrusch-zf
Copy link
Contributor Author

very neat initiative. You could, however, also replace occurences of "[kK]ey[cC]loack" in 6 files.

Good catch! I will do that 😉

@stephanbcbauer
Copy link
Member

very neat initiative. You could, however, also replace occurences of "[kK]ey[cC]loack" in 6 files.

Good catch! I will do that 😉

Can you manage this before merging? ;)

@florianrusch
Copy link

florianrusch commented Dec 24, 2023

I will fix the typos after my vacation ;-) Just let's keep this PR open until I fixed them as well.

@florianrusch-zf florianrusch-zf dismissed stale reviews from FaGru3n and tomaszbarwicki via a71153e January 2, 2024 09:09
@florianrusch-zf florianrusch-zf force-pushed the fix/518-fix-keycloak-spelling branch from 44f8e85 to a71153e Compare January 2, 2024 09:09
@florianrusch-zf florianrusch-zf changed the title fix: replace KeyCloak with Keycloak fix: replace [kK]ey[cC]loac?k with Keycloak Jan 2, 2024
@florianrusch-zf florianrusch-zf changed the title fix: replace [kK]ey[cC]loac?k with Keycloak fix: replace [kK]ey[cC]loac?k with [kK]eycloak Jan 2, 2024
@florianrusch-zf
Copy link
Contributor Author

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:

  • docs-kit/kits/Data Chein Kit/page_software-operation-view.md:215
  • docs-kits_versioned_docs/version-23.09/kits/Data Chain Kit/page_software-operation-view.md:207
  • docs-kits_versioned_docs/version-23.12/kits/Data Chain Kit/page_software-operation-view.md:215
  • docs-kits_versioned_docs/version-3.1.0/kits/Data Chain Kit/page_software-operation-view.md:207

Copy link
Contributor

@drcgjung drcgjung left a comment

Choose a reason for hiding this comment

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

Perfect.

FaGru3n
FaGru3n previously approved these changes Jan 8, 2024
Copy link
Contributor

@FaGru3n FaGru3n left a comment

Choose a reason for hiding this comment

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

LGTM

@carslen
Copy link
Contributor

carslen commented Jan 23, 2024

Wanted to merge PR, but it has merge conflicts. Please resolve merge conflicts @florianrusch-zf

@stephanbcbauer
Copy link
Member

@florianrusch, @carslen I could take over this PR, if needed

@florianrusch-zf
Copy link
Contributor Author

florianrusch-zf commented Jan 25, 2024 via email

Copy link
Member

@stephanbcbauer stephanbcbauer left a 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

@florianrusch-zf florianrusch-zf self-assigned this Jan 26, 2024
@florianrusch-zf florianrusch-zf requested review from FaGru3n and removed request for manojava-gk January 26, 2024 06:19
@carslen carslen merged commit 01e6616 into eclipse-tractusx:main Jan 26, 2024
4 checks passed
@florianrusch-zf florianrusch-zf deleted the fix/518-fix-keycloak-spelling branch April 2, 2024 03:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Replace KeyCloak with Keycloak
7 participants