Skip to content
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

Fix and extend Haddock documentation #89

Merged
merged 2 commits into from
Nov 14, 2019

Conversation

fendor
Copy link
Collaborator

@fendor fendor commented Nov 14, 2019

No description provided.

@fendor fendor mentioned this pull request Nov 14, 2019
14 tasks
@@ -57,10 +57,10 @@ withGhcT body = do

----------------------------------------------------------------

-- | Initialise a GHC session by loading a given file into a given cradle.
-- | Initialize a GHC session by loading a given file into a given cradle.
Copy link
Collaborator

Choose a reason for hiding this comment

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

😱

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Hehe, I actually wanted to ask: Are we using GB or US english here? Function names are US but most comments are GB

| CradleNone -- ^ No attempt was made to load the cradle
-- | Result of an attempt to set up a GHC session for a 'Cradle'.
-- This is the go-to error handling mechanism. When possible, this
-- should be preferd over throwing exceptions.
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
-- should be preferd over throwing exceptions.
-- should be preferred over throwing exceptions.

@mpickering mpickering merged commit b37bcb9 into haskell:master Nov 14, 2019
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.

None yet

2 participants