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.

Sign up for GitHub

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

Jump to bottom

Fix some typos #27936

Merged
merged 2 commits into from
Jan 28, 2019
Merged

Fix some typos #27936

merged 2 commits into from
Jan 28, 2019

Conversation

Cyrano7
Copy link
Member

@Cyrano7 Cyrano7 commented Jan 28, 2019 •
edited by ZhilkinSerg
Loading

Summary

SUMMARY: I18N "Fix some minor spacing and typos"

@MetroidHunter
Copy link
Contributor

It looks like you added double spaces between sentences in some descriptions. Was that intentional?

@KorGgenT
Copy link
Member

you have an invalid summary line.
also, i am wondering if you ran these files through the web linter? if so, i think some of these files are in json_blacklist and could stand to be removed from that file

@ZhilkinSerg ZhilkinSerg added Translation I18n [JSON] Changes (can be) made in JSON labels Jan 28, 2019
@ZhilkinSerg
Copy link
Contributor

Jenkins rebuild.

2 similar comments
@ZhilkinSerg
Copy link
Contributor

Jenkins rebuild.

@ZhilkinSerg
Copy link
Contributor

Jenkins rebuild.

@ZhilkinSerg ZhilkinSerg merged commit 9782187 into CleverRaven:master Jan 28, 2019
@Cyrano7 Cyrano7 deleted the Typos branch January 28, 2019 09:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
[JSON] Changes (can be) made in JSON Translation I18n
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@Cyrano7 @MetroidHunter @KorGgenT @ZhilkinSerg