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

Suggested modification to first exercise #986

Open
wants to merge 2 commits into
base: dev
Choose a base branch
from

Conversation

KevinDCarlson
Copy link

Fixes #960 by handing the student the type declaration. Starts with an explicitly wrong answer to avoid ?s.

@KevinDCarlson KevinDCarlson changed the title Suggested modifcation to first exercise Suggested modification to first exercise Apr 29, 2024
src/plfa/part1/Naturals.lagda.md Outdated Show resolved Hide resolved
@@ -75,14 +75,14 @@ successor of two; and so on.

#### Exercise `seven` (practice) {#seven}

Write out `7` in longhand.
Write out `7` in longhand. The suggestion below loads but is, of course, incorrect.
Copy link
Collaborator

Choose a reason for hiding this comment

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

Why do you want to avoid ??

Copy link
Author

Choose a reason for hiding this comment

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

Because at this point, the reader hasn't learned how to deal with ?--the walkthrough of that is a bit later.

Copy link
Member

Choose a reason for hiding this comment

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

Actually, holes are never used in the executable text, because Agda gets stroppy if you try to import a module that contains a hole.

Copy link
Member

@wadler wadler left a comment

Choose a reason for hiding this comment

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

Thanks!

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.

First exercise too advanced?
3 participants