[Swift 6]: Update Expert mixolgist and its concept #833
Conversation
|
This PR touches files which potentially affect the outcome of the tests of an exercise. This will cause all students' solutions to affected exercises to be re-tested. If this PR does not affect the result of the test (or, for example, adds an edge case that is not worth rerunning all tests for), please add the following to the merge-commit message which will stops student's tests from re-running. Please copy-paste to avoid typos. For more information, refer to the documentation. If you are unsure whether to add the message or not, please ping |
|
Hello 👋 Thanks for your PR. This repo does not currently have dedicated maintainers. Our cross-track maintainers team will attempt to review and merge your PR, but it will likely take longer for your PR to be reviewed. If you enjoy contributing to Exercism and have a track-record of doing so successfully, you might like to become an Exercism maintainer for this track. Please feel free to ask any questions, or chat to us about anything to do with this PR or the reviewing process on the Exercism forum. (cc @exercism/cross-track-maintainers) |
| ``` | ||
|
|
||
| However, if we label the outer loop, calling it `mainLoop`, and specify continuing _that_ loop when the current `letter` is "e" then any non-vowel letters following the "e" are not printed, nor is the trailing line of asterisks and newline. | ||
| ## Fallthrough |
There was a problem hiding this comment.
| ## Fallthrough | |
| ## Fall through |
There was a problem hiding this comment.
If you mean the keyword, it should be lowercase and not titlecase.
There was a problem hiding this comment.
The swift docs uses Fallthrough as one word and with title: https://docs.swift.org/swift-book/documentation/the-swift-programming-language/controlflow/#Fallthrough
| @@ -145,15 +145,13 @@ | |||
| "status": "active" | |||
| }, | |||
| { | |||
| "slug": "master-mixologist", | |||
| "name": "Master Mixologist", | |||
| "slug": "expert-mixologist", | |||
There was a problem hiding this comment.
Is it okay to change the slug? Would Jeremy need to change something in the database to link the old and new slugs?
There was a problem hiding this comment.
Co-authored-by: Isaac Good <[email protected]>
Co-authored-by: Isaac Good <[email protected]>
Co-authored-by: Isaac Good <[email protected]>
Co-authored-by: Isaac Good <[email protected]>
Co-authored-by: Isaac Good <[email protected]>
Co-authored-by: Isaac Good <[email protected]>
Co-authored-by: András B Nagy <[email protected]>
Co-authored-by: András B Nagy <[email protected]>
Co-authored-by: András B Nagy <[email protected]>
Co-authored-by: András B Nagy <[email protected]>
Co-authored-by: András B Nagy <[email protected]>
| Swift has a set of statements that can be used to alter the normal control flow of loops. | ||
| These are known as control transfer statements. | ||
| You have already seen `return`; this concept will introduce other, which include `continue`, `break`, and labels. | ||
| We won't cover `fallthrough` in this concept, but you can read about it in the [Swift book][fallthrough]. |
There was a problem hiding this comment.
Is Fallthrough covered? I see it below.
| While loop's condition is evaluated before the loop body is executed; if the condition is false, the loop body will not be executed at all. | ||
| while-loops in Swift have the following structure: |
There was a problem hiding this comment.
One sentence starts with "While loop's" and the next with "while-loops". It would be good to make these consistent.
This renames the exercise master mixologist to expert mixologist and also renames "while loops" to "while and repeat loops"