Skip to content

📜 Hackathon content update - Ruth's PR #6498

Merged
jpelay merged 56 commits intomainfrom
Hackathon-Content-Update
Mar 11, 2026
Merged

📜 Hackathon content update - Ruth's PR #6498
jpelay merged 56 commits intomainfrom
Hackathon-Content-Update

Conversation

@wgzwemer
Copy link
Collaborator

I have fixed the explanation of new things in level 13-15.
Level 13:

  • Added some explantion for 'print'.
  • Made explanation for 'ask'
  • Made a separate explanation for if with '<, >, and ==' and another explanation for operators '<=', '>=' and '!='

Level 14:

  • Put the explanation of 'for' with list here instead of level 13.
  • Added explanation on 'list' and 'random'
  • Stated clearly that not all is 'real Python'

In level 15:

  • Started on explaining 'functions' in 3 parts
  • Still quite some explanations for the functions have to be added.

Fixes: #6456

@MarleenGilsing
Copy link
Collaborator

Nice job Ruth, I'm turning your PR into a draft so that Jesus knows it's not ready to be merged yet.

@MarleenGilsing MarleenGilsing changed the title 📜 - Hackathon content update 📜 Hackathon content update - Ruth's PR Feb 6, 2026
@MarleenGilsing
Copy link
Collaborator

MarleenGilsing commented Feb 6, 2026

I'm trying not to touch level 13 and up now, so we don't get any merge conflicts, so I've got a small job for you. I have found a weird mistake in level 7. Somehow I have put the adventure 'quizmaster' in level 7, where it definitely doesn't belong because we don't use <> yet (i think because we were planning on using the words greater than and less than, but we're gonna let that ship sail) anyway... I think it's better to change it on your PR, so level 13+ won't get any merge conflicts, hopefully...

Could you:

  • Change quizmaster from level 7 to level 13 in the english file.
  • Make the content level 13 proof (so the right syntax in the example code).
  • Check whether all the keywords are marked as such with the curly braces, I recieved a comment that this was not the case in this adventure.
  • update the Dutch quizmaster adventure as well
  • Remove the quizmaster adventure from the other languages (because fixing the syntax in other languages is probably too hard, unless you feel upto it).
  • Add Quizmaster to level 13 in hedy_content.py and remove it from level 7

Let me know if you have any questions!

@wgzwemer
Copy link
Collaborator Author

wgzwemer commented Feb 7, 2026

@MarleenGilsing

Done it!

The quizmaster was only there in English actually. So I added it in Dutch and the rest can be done by translators.

@MarleenGilsing MarleenGilsing marked this pull request as ready for review March 6, 2026 08:17
@jpelay jpelay merged commit d60fe71 into main Mar 11, 2026
6 of 9 checks passed
@jpelay jpelay deleted the Hackathon-Content-Update branch March 11, 2026 15:00
@github-project-automation github-project-automation bot moved this from In Progress to Done in Hedy organization board Mar 11, 2026
@mergify
Copy link
Contributor

mergify bot commented Mar 11, 2026

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).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done
Status: Review for relevance

Development

Successfully merging this pull request may close these issues.

📜Take a good look at level 13-16

3 participants