Skip to content
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

Avoid setting encapsulation challenge too early #579

Merged
merged 1 commit into from
Dec 28, 2021

Conversation

alex-ball
Copy link
Contributor

In episode 16, the panel "Encapsulation of an If/Print Block" asks the learner to turn the if-block into a function a total of three times:

  • "Please re-write the code so that the if-block is folded into a function."
  • "What function definition will make it functional?"
  • "Create a function definition..."

Only the last of these is needed. The first one is particularly unhelpful as it comes before the task is properly defined.

This pull request removes the duplication, and instead provides a potential rationale for wanting to do the task in the first place. (It also removes some trailing spaces.)

In episode 16, the panel "Encapsulation of an If/Print Block" asks the
learner to turn the if-block into a function a total of three times:

- "Please re-write the code so that the if-block is folded into a
  function."
- "What function definition will make it functional?"
- "Create a function definition..."

Only the last of these is needed. The first one is particularly
unhelpful as it comes before the task is properly defined.
Copy link
Member

@alee alee left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for catching this!

@alee alee merged commit e61632a into swcarpentry:gh-pages Dec 28, 2021
@alex-ball alex-ball deleted the patch-3 branch January 4, 2022 09:28
tobyhodges pushed a commit that referenced this pull request Apr 17, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants