-
Notifications
You must be signed in to change notification settings - Fork 84
Findings from the essentials course #550
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
Side comment: the print helper is handy but may be frightening for beginners. |
The final exercise (loospRefsAuto) was somehow redundant with the third one (functions). Not sure how to improve this. |
It would help to have more explicit recommendations on the required minimal version of g++ and clang++... which would require an easy way to check that a given version is ok for all exercises ? Perhaps already done in the CI. |
I see (only now) that there are many installation instructions in the exercices/README. Fine ! Is it somehow up to date ? Probably it deserves a dedicated file that we may review regularly. |
4457496
to
46f4e65
Compare
The question was about boolean logic, and not precedences.
Thanks to Oliver for this idea!
46f4e65
to
3f31988
Compare
3f31988
to
ac47325
Compare
I added a NOTE that it's not part of the exercise.
The CI already checks that exercises compile with the respective versions. You are right that we have to remember to update this part cpluspluscourse/exercises/README.md Lines 23 to 28 in cde47b7
when we move to C++23. The cpluspluscourse/exercises/check_setup.sh Lines 38 to 39 in cde47b7
|
The warning box was perceived as too strong. We don't want to discourage people from templates.
Running list of improvements from the 12th instalment