Skip to content

Fix haskell-docs invocation. #573

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 1 commit into from
Apr 8, 2015
Merged

Fix haskell-docs invocation. #573

merged 1 commit into from
Apr 8, 2015

Conversation

gracjan
Copy link
Contributor

@gracjan gracjan commented Apr 7, 2015

Fixes #535.

@gracjan
Copy link
Contributor Author

gracjan commented Apr 7, 2015

@lukehoersten: can you review and merge?

@lukehoersten
Copy link
Member

Tested with:

test = toUpper <$> Just "hey"

and got the following failure (including *Messages* context):

Not in scope: ‘toUpper’
Identifier `toUpper' not in scope, choose module to import? (y or n) y
Invalid face reference: ido-first-match-face [106 times]
Haskell process command errored with: (wrong-type-argument listp t)

I don't think this patch fixes the issue.

@gracjan
Copy link
Contributor Author

gracjan commented Apr 8, 2015 via email

@lukehoersten
Copy link
Member

I only have default ido. ido-first-match-face isn't listed as a valid function or variable on my emacs.

Unfortunately I can't test if the <$> doc escape search problem is fixed because the other issue would cover it up.

@lukehoersten
Copy link
Member

I tested with a clean emacs config and everything works fine. I tried and it works fine with ido in a clean config as well. Something else I have in my config is conflicting. We'll have to track it down but the <$> escape problem is fixed for sure.

lukehoersten added a commit that referenced this pull request Apr 8, 2015
@lukehoersten lukehoersten merged commit 70cdab6 into haskell:master Apr 8, 2015
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.

Import suggester always fails
2 participants