Skip to content

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