Update basics/about.md
#708
Conversation
|
For @SleeplessByte. I can't assign the reviewer (as a non-maintainer here). |
| Functions can have parameters with default values. | ||
| These values will be used if they are omitted where the function is invoked: | ||
|
|
||
| ```kotlin | ||
| fun run() {} | ||
| fun ping(host: String = "localhost") { | ||
| println("PING --> $host") | ||
| } |
There was a problem hiding this comment.
In JS this is part of functions concept. I don't mind having it here, but if there will be separate concept, should move there. You can embed a link to a concept. Let me know if you don't know how to.
There was a problem hiding this comment.
I didn't include this at first, for the exact reasons you give (there will be a Functions concept, though I've not written anything yet).
Unfortunately, the optional argument thing is baked into the current version of Lasagna. We could rewrite that, but it may be a backwards step for something already merged?
There was a problem hiding this comment.
Let's leave it for now, we build the syllabus, and then we do a second (quick) run together making changes like that?
There was a problem hiding this comment.
I suspect it won't be the only thing we change later. A syllabus may need iterations!
There was a problem hiding this comment.
Haha exactly. In JS (and TS later) we have many iterations.
Co-authored-by: Derk-Jan Karrenbeld <derk-jan+github@karrenbeld.info>
Co-authored-by: Derk-Jan Karrenbeld <derk-jan+github@karrenbeld.info>
Co-authored-by: Derk-Jan Karrenbeld <derk-jan+github@karrenbeld.info>
Added a few extra lines.
removed double "for"
|
@colinleach good to merge? |
|
It's as good as I can get it, for now... |
2 participants
A small first PR, just touching one file.
In the merged version of this concept, the
introduction.mdis longer and more detailed than theabout.md. The opposite should be true!This is a new
about.md, based on the Introduction but expanded a little.