Skip to content

compile the code snippets in tutorial/api-type.(md/lhs) #37

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

Merged
merged 7 commits into from
Jan 18, 2016

Conversation

soenkehahn
Copy link
Contributor

Still working on converting all the files. The testing machinery is ready for review.


set -o errexit

cabal exec -- ghc -outputdir build-output ../tutorial/api-type.lhs
Copy link
Member

Choose a reason for hiding this comment

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

I imagine you know the other files still need to be added?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yup.

@soenkehahn
Copy link
Contributor Author

I realize that in the tutorial there's lots of code quoted from the servant source. That of course clashes with the things imported from servant. Not sure how to deal with that nicely. For now I just keep it as unchecked code blobs in there.

@soenkehahn
Copy link
Contributor Author

I'm wondering whether all imports in the tutorial should have explicit import lists. Any opinions?

(@jkarni, @alpmestan, @fizruk)

@jkarni
Copy link
Member

jkarni commented Jan 18, 2016

I'd say so, but we can do that as part of the 0.5 rewrite (or separately altogether) rather than as part of this.

@fizruk
Copy link
Member

fizruk commented Jan 18, 2016

I am not strongly inclined to having explicit imports.
I also agree than we can clean this up later.

@alpmestan
Copy link
Contributor

+1, let's worry about that later.

@jkarni
Copy link
Member

jkarni commented Jan 18, 2016

LGTM

@soenkehahn soenkehahn merged commit ebb0fc4 into hakyll Jan 18, 2016
@soenkehahn soenkehahn deleted the compile-examples branch January 24, 2016 23:10
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.

4 participants