-
-
Notifications
You must be signed in to change notification settings - Fork 72
[Lasagna] revision to existing concept exercise #780
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
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 |
Update: one test is failing on Julia 1.6, the current LTS version, but passes on 1.10, the current latest release. I'll add a guard clause to this test and push a revision. FYI - 1.6 will probably be dropped by the end of this year, when 1.11 is released and 1.10 becomes LTS. |
|
||
julia> -1234 | ||
-1234 | ||
Global variables, created outside any function, are: |
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.
The heading refers to constants but we don't mention constant until later. Perhaps we should introduce what a constant is first and then discuss global variables to show why a constant is preferred.
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.
Agree
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.
Extensively reworded. What do you think of the new attempt?
Co-authored-by: András B Nagy <[email protected]>
Co-authored-by: András B Nagy <[email protected]>
There's stuff I need to do for the next couple of hours. I'll go through your suggestions again later today and push the changes. This exercise has been rumbling around for about 5 years. so it would be good to finally get it right! |
Fixing the requested change I missed in the previous commit.
Once we agree on the |
I'm trying to keep this first exercise to a minimum, so leaving out implicit multiplication was deliberate. I'm thinking of adding a I haven't made progress on this in the last few days, other than reading around some other tracks. Blame some passing infection and associated fever (the sort of non-specific thing that epidemiologists lump together as ILI (Influenza-Like Infection) and normal people might call "something going around"). Not an aid to clear thinking. |
I hope you feel better soon! |
|
||
julia> -16 | ||
-16 | ||
These mostly work conventionally: |
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.
This sentence is a little confusing since we don’t detail what’s not conventional in the next few lines.
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 update tomorrow.
Co-authored-by: András B Nagy <[email protected]>
oops... Co-authored-by: András B Nagy <[email protected]>
Reading through all this again, months later, I think all the comments are addressed? You can approve when you're happy, but please don't try to merge yet - it will complain about the missing |
Oops, no. I had (inevitably) missed one requested change. |
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.
This semi-blank slug seems appropriate for the intentions of the exercise. Everything else also look okay to me
Yes, an exercise to learn how to add functions is the exception to the rule we discussed earlier! |
For discussion on the forum
This revision (the most recent of many):
introduction.md
to meet Exercism guidelines.config.sys
to thebasics
concept, instead of 3 older ones.The old documentation isn't wasted: I moved some to
basics/about.md
and some to later concepts.Including the
introduction.md.tpl
was my mistake. It should probably be removed now we no longer need to combine bits from 3 concepts.