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

locale(ko-kr): update Korean localization #894

Merged
merged 6 commits into from
Jun 12, 2022
Merged

locale(ko-kr): update Korean localization #894

merged 6 commits into from
Jun 12, 2022

Conversation

ZISQO
Copy link
Contributor

@ZISQO ZISQO commented Jun 12, 2022

so terrible sorry about you because of here is a new edition even you has been accepted the pull request. (12th june 2022 @ GMT+9)

so terrible sorry about you because of here is a new edition even you has been accepted the pull request. (12th june 2022 @ GMT+9)
@ZISQO
Copy link
Contributor Author

ZISQO commented Jun 12, 2022

i have no idea of error on Code style check / Prettier (pull_request_target) with 1 error. do you have something for fixing it?

@dw-0 dw-0 changed the title Update ko-kr.json locale(ko-kr): update Korean localization Jun 12, 2022
Signed-off-by: Dominik Willner <[email protected]>
@dw-0
Copy link
Member

dw-0 commented Jun 12, 2022

i have no idea of error on Code style check / Prettier (pull_request_target) with 1 error. do you have something for fixing it?

The indentation level you committed that file with was 2 instead of 4 spaces.
I pushed a commit to this PR that lints and fixes that issue.

dw-0
dw-0 previously approved these changes Jun 12, 2022
@ZISQO
Copy link
Contributor Author

ZISQO commented Jun 12, 2022

I pushed a commit to this PR that lints and fixes that issue.

thanks a lot and i was changed the settings of prettier with JSON on vsc! have nice day!

@meteyou meteyou changed the base branch from develop to master June 12, 2022 20:20
@meteyou meteyou merged commit 5029f4d into mainsail-crew:master Jun 12, 2022
meteyou added a commit that referenced this pull request Jun 12, 2022
Co-authored-by: th33xitus <[email protected]>
Co-authored-by: Stefan Dej <[email protected]>
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.

3 participants