-
-
Notifications
You must be signed in to change notification settings - Fork 69
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
stage-heralding: update instructions #600
Comments
Fwiw, it's given in the hints. The point of the exercise isn't to write the regex. I've done some testing in the past and it only slightly overshot the 10min goal, most solved it in 10-15min. |
Hmmm, maybe it's me 🤷♂️ I'd like someone else to try it too. BTW: I don't think the hints are the right location, as to me it is more of a requirement |
Sorry, didn't mean to discredit your experience with the exercise. I wanted to point out that writing the regex isn't really meant to be a required part of the exercise unless they want to do it, and when excluding that, the target of 10ish min for an experienced user is realistic. We could also consider wrapping the regex in a spoiler in the instructions instead of in the hints. |
Oh I misread. The regex is in the hints? I missed that. Maybe this it would then be enough to mention this more explicitly in the instructions. |
When doing the exercise, I felt that maybe this exercise could be simplified a little bit, as the regex one has to write is quite big.
The pronouns translation is not mentioned anywhere in the instructions (maybe even omit this or rephrase the card to not have to do this translation)
Maybe split up the instructions into smaller tasks, to have the student focus on things on at a time (e.g. by first having tasks to extract individual bits of information)
The text was updated successfully, but these errors were encountered: