-
Notifications
You must be signed in to change notification settings - Fork 14.4k
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
es: Revision and translation #222
Conversation
Observaciones hechas hasta la línea 156. En adelante no ha sido revisado.
#121 Finally here's my PR. |
@@ -1,4 +1,4 @@ | |||
[ Languages: [English](README.md), [Español](README-es.md), [한국어](README-ko.md), [Português](README-pt.md), [Русский](README-ru.md), [Slovenščina](README-sl.md), [中文](README-zh.md) ] | |||
[ Idiomas: [English](README.md), [Español](README-es.md), [한국어](README-ko.md), [Português](README-pt.md), [Русский](README-ru.md), [Slovenščina](README-sl.md), [中文](README-zh.md) ] |
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.
@jlevy, translate Languages
in each language looks a good idea. What do you think?
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.
Sure. The only problem is, I have to update all of them when we add new languages, so I may not preserve this just out of laziness...
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.
@aaossa Keep Language for now, we see this in a separate Pull Request.
Thanks
@aaossa Great Job. I did some observations, please check. |
Thanks @ceoaliongroo . I'll make a commit in a few hours. Tomorrow i'll check the remaining lines. EDIT: Should we use |
Some suggestions and questions: 206: Also, should we use |
Just a note -- let's keep translations semantically identical. As in don't make changes like
above. This will cause divergence/confusion across the languages. (And in this particular case, it's also incorrect, as you don't press them at once.) Thanks! Really appreciate all the work making the Spanish one better! |
|
||
- Para archivos Excel o CSV, [csvkit](https://github.com/onyxfish/csvkit) provee `in2csv`, `csvcut`, `csvjoin`, `csvgrep`, etc. | ||
|
||
- Para Amazon S3, [`s3cmd`](https://github.com/s3tools/s3cmd) es conveniente y [`s4cmd`](https://github.com/bloomreach/s4cmd) es el mas rápido. Hecho por Amazon [`aws`](https://github.com/aws/aws-cli) es esencial para otras tareas relacionadas al AWS. | ||
|
||
- Conocer acerca `sort` y `uniq`, incluyendo opciones de uniq `-u` y `-d` -- ver unas líneas más abajo. | ||
- Conocer acerca de `sort` y `uniq`, incluyendo las opciones de uniq `-u` y `-d` -- ver one-liners más abajo. |
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.
Change:
one-liners
=> [one-liners](https://github.com/jlevy/the-art-of-command-line/blob/master/README-es.md#one-liners)
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.
Done.
@aaossa Pretty good the corrections....Awsome!
Answer to you: about I open the issue #226 to verify this apart the point may need some discussion, and maybe affect other language |
Thanks again @ceoaliongroo , the next commit will include your suggestions except for 185 and 230, i'll wait your answer on those. I'll keep an eye on that issue. Maybe when it is decided a PR will fix it. |
Observations until line 156.