-
Notifications
You must be signed in to change notification settings - Fork 347
merged hi2 (https://github.com/errge/hi2). #314
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
Conversation
@kuribas: does this merge replace haskell-indentation with hi2? In case you did not see results:
|
@purcell: this looks like old version of hi2, not relevant anymore. This particular pull request can be closed but we should think what to do about hi2 as it is the future of indentation. |
It's not an old version of hi2, I merged the changes of hi2 with the current haskell-indentation.el. |
@kuribas: Great! I just saw that this pull request is from 5 Aug 2014 while last changes to nilcons/hi2 were done on 5 Oct 2014. Do you intent to develop hi2 as separate package or as part of haskell-mode? |
@gracjan: no, I think it's better to remove the hi2 fork, and develop haskell-indentation.el as part of haskell-mode. |
@kuribas: Am I correct that haskell-indentation was an evolution of haskell-indent in the same sense as hi2 was evolution of haskell-indentation? (This predates my involvement here so I lack knowledge of history). |
No, haskell-indentation was written from scratch. |
@kuribas: is there anything provided by haskell-indent that is missing in haskell-indentation (hi2)? |
Yes, the two are totally different modes, with different features and keybindings. |
@kuribas: that is unfortunate, because haskell-indent does not seem to be maintained. |
I wrote haskell-indentation because I didn't like editing with haskell-indent, so I cannot help you here, sorry. |
@kuribas: yeah, nobody likes haskell-indent. I hoped that haskell-indentation subsumes haskell-indent so we can solve the usual 'which haskell indentation mode should I use' in favor of haskel-indentation. |
@gracjan @kuribas may I remind you that it's not so clear cut which indentation mode is the single most favorite of everone? See https://docs.google.com/forms/d/1FzWbzGm6odYWxJZcU3GFHlS3lVFTBOI1-M1c87CjOFg/viewanalytics where all 3 indentation modes were more or less |
Preference is on Why |
@hvr: also note that I've worked a lot on |
@gracjan I don't think many people know about hi2, that's why I wanted to push the changes to haskell-indentation. Most people just use it because it comes with haskell-mode I guess. As @hvr mentioned, I wouldn't recommend on removing haskell-indent. But I'd use haskell-indentation as default. Of course I am terribly biased :-) I know that haskell-indentation sometimes fails on syntax-extensions. That would be an area of improvement. I'd rather see people sending bug requests than rejecting haskell-indentation completely. |
Yes, |
@kuribas: I guess a rebase over master would be needed to get a clean compile. |
Came here to close this as suggested, but if I've misunderstood, feel free to reopen. |
Yes, a bit too fast :) |
No problem. |
This will use the modification from hi2. It highlights indentation columns, and binds shift-tab to indent to the left, instead of backspace. It's probably a good idea to warn users that the keybindings have changed.