-
Notifications
You must be signed in to change notification settings - Fork 1k
Italian translation added #453
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
Conversation
Italian translation for the tutorial added. It is a good opportunity to review what I previously did in 2009 - http://bit.ly/1EJepIK
@cunei, might you be interested in reviewing this? (or could you suggest an Italian-speaking reviewer?) |
@SethTisue, I can review this text. An additional reviewer could be @dotta |
Per compilare l'esempio useremo `scalac`, il compilatore Scala. | ||
`scalac` lavora come la maggior parte dei compilatori: prende un file | ||
sorgente come argomento, eventuali opzioni e produce uno o più object | ||
file come output. Gli object file sono gli standard file delle classi |
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.
s/object file/class file
Also considering that in the next paragraph class file
is used already.
@indaco Great stuff! I had a few comments, but overall it looks pretty good to me! @SethTisue I've reviewed a good chunk of it, and will try to finish it off next week. However, I haven't read the original text, so mine is not a translation review (just pointing it out, not sure you guys care). |
diversi tipi di contenitori sono basati sugli alberi, come gli alberi | ||
red-black. | ||
|
||
Esamineremo ora come gli alberi sono rappresentati e manipolati in Scala |
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.
Instead of "sono", I think it's better to say "possono essere".
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.
I'll fix it
thanks Mirco! I'd suggest a new ticket with your feedback on the content. that feedback is valuable. this PR specifically is mergeable as long as there aren't glaring translation mistakes. |
valore al nome di una variabile. L’environment `{ x -> 5 }` | ||
mostrato sopra può essere semplicemente scritto in Scala come: | ||
|
||
{ case "x" => 5 } |
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.
Why using a partial function and not a plain function?
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.
Aaah, I see, just reading what comes next. OK then.
@SethTisue Alright, I've just decided to do it all in one pass otherwise I'll forget again :-) I'd definitely say the PR is good to merge, no glaring mistakes in the text itself, but I don't know how much it differs from the original text. Feel free to create tickets as needed. |
By the way, @cunei, you are such a slacker! :-P |
Although most of your comments are valuable I just want to clarify that I I simply use the original english tutorial, maybe it's time to update the original one :-) in order to be updated with the latest scala and java 8 features. What's your thoughts? |
@indaco Ciao Mirco! No worries, I was asked to review the text and did so, but also pointed out that I didn't read the original :-) I'm actually surprised @SethTisue hasn't merged this PR yet. I think it'd be great to update the original text. If you have time, I'm sure your contribution would be highly valued! By the way, thanks again - a lot - for the really great translation! |
I think it was just an oversight that this wasn't merged a while ago. thank you, @indaco. yeah, and yeah, the content just gradually gets more and more outdated. :-\ |
oh, here is @SethTisue :-) |
jinx! |
:-) Thank you |
whoops, this broke the site build: https://travis-ci.org/scala/scala.github.com/builds/120106186 I'm investigating. |
oh no! |
no worries, easy fix: b19591b this is showing up on the site now: there is an "Italiano" link at http://docs.scala-lang.org/tutorials/scala-for-java-programmers.html (fwiw, note that Travis would normally catch a bug like this pre-merge, but that didn't happen here because this PR predates the addition of Travis to the repo.) |
Added Italian translation for the tutorial. It is a good opportunity to review what I previously did in 2009 - http://bit.ly/1EJepIK