-
Notifications
You must be signed in to change notification settings - Fork 210
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Use
Prelude/
prefix in tutorial examples (#184)
Fixes #59 The recent refactor of the Dhall grammar made `List/map` no longer a valid identifier because the parser interprets it as `List /map`. The grammar can't be easily fixed to accept this variable name because: * the normal way you'd fix this is to use a lexer with a longest-match rule * however, Dhall cannot use a lexer due to nested block comments and string interpolation So this sort of identifier name is no longer supported under the new grammar. The work-around is to prefix the name with something that is not a built-in, such as `Prelude/List/map`. This change updates the tutorial to use the `Prelude/` prefix work-around so that people don't accidentally run into this issue by following the tutorial.
- Loading branch information
1 parent
3a625d7
commit 834c1b6
Showing
1 changed file
with
32 additions
and
32 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
834c1b6
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Great, thanks for clarifying this!
834c1b6
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You're welcome! ☺