-
Notifications
You must be signed in to change notification settings - Fork 285
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
📜 Fix quizzes in all languages #5479
Conversation
Correcting a few mistakes in the quiz
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.
Thanks for this PR and welcome to Hedy @pepa65!
This issue will probably also exist in other yamls (which are a copy of the en file) Are you able to also make these changes in other languages where needed 🙏?
All kinds of problems just looking at Quiz 16 question 10... This whole translation file cannot be relied upon without review.
Used `|-` for the Code listings
OK, done..! |
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.
Thanks for the extra work on the other languages!! I fixed a small issue (a weird copy-paste error I think?) and we should be good to go now
Thank you for contributing! Your pull request is now going on the merge train (choo choo! Do not click update from main anymore, and be sure to allow changes to be pushed to your fork). |
Correcting a few mistakes in the Quiz section.
Fixes #5478
Checklist