Skip to content

Add Prim docs to the library, resolves #2494#2498

Merged
paf31 merged 3 commits intomasterfrom
include-prim-docs
Dec 24, 2016
Merged

Add Prim docs to the library, resolves #2494#2498
paf31 merged 3 commits intomasterfrom
include-prim-docs

Conversation

@hdgarrood
Copy link
Copy Markdown
Contributor

No description provided.

Copy link
Copy Markdown
Contributor

@paf31 paf31 left a comment

Choose a reason for hiding this comment

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

Thanks, looks great!

main :: IO ()
main = do
putStrLn "Test that there are no bottoms hiding in primDocsModule"
seq (T.pack (D.runDocs (D.modulesAsMarkdown [D.primDocsModule]))) (return ())
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

You could use Control.Exception.evaluate here.

, "in a custom type error."
, ""
, "For more information, see"
, "[the Custom Type Errors guide](https://github.com/paf31/24-days-of-purescript-2016/blob/master/21.markdown)."
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I wonder if we should just create a placeholder page in the docs repo for now instead of linking to "24 Days" which is more likely to become out of date.

@paf31 paf31 merged commit a1e22ec into master Dec 24, 2016
@paf31 paf31 deleted the include-prim-docs branch December 24, 2016 02:55
@paf31
Copy link
Copy Markdown
Contributor

paf31 commented Dec 24, 2016

👍 Great, thanks!

@hdgarrood
Copy link
Copy Markdown
Contributor Author

I think @garyb's point about Partial not always throwing is a good one, I'll try to get around to addressing that soon.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants