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

Update materials to consistently use title case #55836

Merged
merged 1 commit into from
Mar 9, 2022

Conversation

Faalagorn
Copy link
Contributor

Summary

None

Purpose of change

Unify names of materials

Describe the solution

Change the materials not following title case to title case

Describe alternatives you've considered

Use lowercase instead of title case, but unless a different convention will be thought of, I used the one most items followed.

Testing

Just a name chance, new material names will be visible on items, especially the

Additional context

Should fix an issue or two in Transifex while at it.

@github-actions github-actions bot added Json astyled astyled PR, label is assigned by github actions json-styled JSON lint passed, label assigned by github actions BasicBuildPassed This PR builds correctly, label assigned by github actions labels Mar 5, 2022
@Faalagorn Faalagorn changed the title Update mateirals to consistently use title case Update materials to consistently use title case Mar 5, 2022
@Maleclypse Maleclypse added the <Bugfix> This is a fix for a bug (or closes open issue) label Mar 6, 2022
@kevingranade kevingranade merged commit e98a360 into CleverRaven:master Mar 9, 2022
@Faalagorn Faalagorn deleted the patch-3 branch March 9, 2022 19:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
astyled astyled PR, label is assigned by github actions BasicBuildPassed This PR builds correctly, label assigned by github actions <Bugfix> This is a fix for a bug (or closes open issue) json-styled JSON lint passed, label assigned by github actions
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants