Skip to content

Allow Natural and Integer literals in binary notation#1355

Merged
Gabriella439 merged 4 commits intodhall-lang:masterfrom
kukimik:binary
Nov 20, 2023
Merged

Allow Natural and Integer literals in binary notation#1355
Gabriella439 merged 4 commits intodhall-lang:masterfrom
kukimik:binary

Conversation

@kukimik
Copy link
Copy Markdown
Contributor

@kukimik kukimik commented Oct 3, 2023

Closes #1215.

Haskell implementation: dhall-lang/dhall-haskell#2540.

@Gabriella439
Copy link
Copy Markdown
Contributor

I'll also help you tie the knot by merging the corresponding Haskell PR so that this will pass the dhall-lang CI

@Gabriella439
Copy link
Copy Markdown
Contributor

Oh wait, sorry, this can actually be fixed locally. I think you need to update the reference implementation in this repository so that the tests pass. In particular, you'll need to make a matching change here:

https://github.com/dhall-lang/dhall-lang/blob/master/standard/Parser.hs

@kukimik
Copy link
Copy Markdown
Contributor Author

kukimik commented Oct 16, 2023

Fixed. Sorry, I missed that somehow. Thank you for pointing that out.

@Gabriella439 Gabriella439 merged commit c6f3265 into dhall-lang:master Nov 20, 2023
@Gabriella439
Copy link
Copy Markdown
Contributor

Thank you for contributing this!

@kukimik kukimik deleted the binary branch November 20, 2023 13:50
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.

Binary literals

2 participants