Skip to content

haskell-load: add haskell-goto-first-error #818

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

Closed
wants to merge 1 commit into from
Closed

haskell-load: add haskell-goto-first-error #818

wants to merge 1 commit into from

Conversation

da-x
Copy link
Contributor

@da-x da-x commented Aug 19, 2015

I usually start to fix stuff from top to bottom. This helps finding the first error.

@gracjan
Copy link
Contributor

gracjan commented Aug 19, 2015

Hi,

Good function. Is it connected to anything?

@da-x
Copy link
Contributor Author

da-x commented Aug 19, 2015

I set up my haskell-goto-* bindings to match those of my flycheck setup, which are different from the haskell-interative-mode defaults, which are M-n and M-p for next and prev respectively ever since 2012. I defer on this new binding for @chrisdone to decide.

@gracjan
Copy link
Contributor

gracjan commented Sep 28, 2015

We can't burden our users to somehow discover that this exists and then bind the keys themselves.

For this PR to be merged we need:

  • user interface element where it is bound (keybinding)
  • documentation in haskell-mode.texi
  • a unit test case

@da-x
Copy link
Contributor Author

da-x commented Oct 2, 2015

Added a binding. However, considering that haskell-mode.texi is very lacking around the haskell-process mode, and that the unit tests of haskell-process around UI are practically nonexistent, it's a bit over my head right now to extend on those. Do what you will :)

@bergey
Copy link
Contributor

bergey commented Jan 14, 2016

Finishing this PR is on my list, too.

@gracjan
Copy link
Contributor

gracjan commented Jan 14, 2016

Good. Thanks.

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.

3 participants