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

Standardise spelling and casing of homeserver, identity server, and integration manager #1782

Merged
merged 6 commits into from
Jul 15, 2021

Conversation

psrpinto
Copy link
Contributor

@psrpinto psrpinto commented Jul 13, 2021

This PR standardises the spelling and casing of the following terms, across the codebase, as per element-hq/element-web/issues/10569:

  • homeserver
  • identity server
  • integration manager

While the issue mentions only user-visible text, this PR changes all instances of the terms, regardless of whether they're user-visible or not. I think there's value in standardising these terms even in non-user-visible text, so I went ahead and applied the changes to all text.

For reference, here's the script I used to find candidates for replacement:
https://gist.github.com/psrpinto/b3787bae212d5d99649b517e2efd4dce

psrpinto added 5 commits July 13, 2021 16:27
Signed-off-by: Paulo Pinto <paulo.pinto@automattic.com>
Signed-off-by: Paulo Pinto <paulo.pinto@automattic.com>
Signed-off-by: Paulo Pinto <paulo.pinto@automattic.com>
Signed-off-by: Paulo Pinto <paulo.pinto@automattic.com>
Signed-off-by: Paulo Pinto <paulo.pinto@automattic.com>
Signed-off-by: Paulo Pinto <paulo.pinto@automattic.com>
@psrpinto psrpinto marked this pull request as ready for review July 13, 2021 18:28
@germain-gg germain-gg requested a review from a team July 14, 2021 09:56
@germain-gg germain-gg merged commit 6098f74 into matrix-org:develop Jul 15, 2021
@dbkr dbkr added the T-Defect label Jul 15, 2021
@psrpinto psrpinto deleted the fix/naming branch July 16, 2021 13:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants