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

show an anti-idiom first #989

Merged
merged 1 commit into from
Mar 21, 2023
Merged

Conversation

DevinBayly
Copy link
Contributor

I think it would be helpful to simplify the intro paragraph to something like: if you want to convert a temperature you could write... if you wanted to convert two temperatures you could copy the line and change on part, and then show code with 10 copied lines. Now say "Does this happen in your code? how can you avoid this?"

Then show the function definition image

Instructions

Thanks for contributing! ❤️

If this contribution is for instructor training, please email the link to this contribution to
checkout@carpentries.org so we can record your progress. You've completed your contribution
step for instructor checkout by submitting this contribution!

Keep in mind that lesson maintainers are volunteers and it may take them some time to
respond to your contribution. Although not all contributions can be incorporated into the lesson
materials, we appreciate your time and effort to improve the curriculum. If you have any questions
about the lesson maintenance process or would like to volunteer your time as a contribution
reviewer, please contact The Carpentries Team at team@carpentries.org.

You may delete these instructions from your comment.

- The Carpentries

I think it would be helpful to simplify the intro paragraph to something like: if you want to convert a temperature you could write... if you  wanted to convert two temperatures you could copy the line and change on part, and then show code with 10 copied lines. Now say "Does this happen in your code? how can you avoid this?" 

Then show the function definition image
@ineelhere ineelhere merged commit 4e1980e into swcarpentry:gh-pages Mar 21, 2023
@ineelhere
Copy link
Member

Merged #989

zkamvar pushed a commit that referenced this pull request Apr 21, 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