-
Notifications
You must be signed in to change notification settings - Fork 218
Remove Optional/fold and Optional/build #1802
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
Conversation
|
I don't understand why the type-inference failure tests that I added fail in this way: Indeed I assume I must have made a mistake somewhere in the parser… |
|
Ah, I had forgotten to update the |
0547a47 to
4e684f2
Compare
4e684f2 to
a0cecc6
Compare
|
Got a hopefully spurious AppVeyor error: Retrying… |
a0cecc6 to
c352e43
Compare
|
I had forgotten to update the |
|
@sjakobi: I think this just requires updating the path in the |
|
Many thanks for the hint @Gabriel439. Turns out we never included the cache file for the poisoned cache test: https://github.com/dhall-lang/dhall-lang/blob/549470527dcc0ee59253708c6ac23c0cee455093/tests/import/cache/dhall/1220618f785ce8f3930a9144398f576f0a992544b51212bc9108c31b4e670dc6ed21 D'oh! |
a46322b to
d32a713
Compare
|
@sjakobi: You're welcome! 🙂 |
…as standardized in dhall-lang/dhall-lang#1014.
d32a713 to
12e4365
Compare
…as standardized in dhall-lang/dhall-lang#1014.
Note that the test suite doesn't completely pass yet since we haven't updated the code to comply with dhall-lang/dhall-lang#980 yet.