-
-
Notifications
You must be signed in to change notification settings - Fork 377
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(es): Typos/grammar review #689
Conversation
Full review of the ES language.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hi, thank you for your contribution.
Sadly there went something wrong with all indentations. I assume your Editor/IDE you use replaced all indentations with spaces with indentations with tabs. We can't merge that this way.
Please make sure your Editor/IDE follows our file formatting.
If you struggle to make the appropriate changes, please let me know.
Replaced all indentations with correct number of spaces.
Hi! I changed the file by replacing the indentations with spaces, as per the original file. Cheers, |
Okay that worked. But now i saw that there is still an issue. You seem to use CRLF as linefeeds. We need LF linefeeds. |
Used VSC to change to LF
Ok, third time's the charm! (maybe, haha) I used VSC to change to LF, but I'm not sure I've done things correctly, cause now I see that I Merged the branch, maybe? |
I think it is okay now. It's hard to tell because a lot of blocks seem to have changed their position inside of that file. |
# Conflicts: # src/locales/es.json
Signed-off-by: Stefan Dej <[email protected]>
Full review of the ES language.