Skip to content

[#264] Support latest Dhall spec#278

Merged
vrom911 merged 2 commits intomasterfrom
chshersh/264-Support-latest-Dhall-spec
May 13, 2020
Merged

[#264] Support latest Dhall spec#278
vrom911 merged 2 commits intomasterfrom
chshersh/264-Support-latest-Dhall-spec

Conversation

@chshersh
Copy link
Copy Markdown
Contributor

Resolves #264

Temporary solution: escape function on manually this time. Shouldn't need to do it after fix in dhall-json.

@chshersh chshersh added the hlint label May 12, 2020
@chshersh chshersh requested a review from vrom911 as a code owner May 12, 2020 15:51
@chshersh chshersh self-assigned this May 12, 2020
Copy link
Copy Markdown
Contributor

@hint-man hint-man bot left a comment

Choose a reason for hiding this comment

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

There is no place for me here... I will choose the truth I like.

test/Spec.hs Outdated

import qualified Control.Applicative as App

foo = App.empty
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

🤔

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Sorry for that, leftover from testing...

@vrom911 vrom911 merged commit 074e163 into master May 13, 2020
@vrom911 vrom911 deleted the chshersh/264-Support-latest-Dhall-spec branch May 13, 2020 08:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Support latest Dhall spec

2 participants