Skip to content

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

Closed
wants to merge 2 commits into from
Closed

merged hi2 (https://github.com/errge/hi2). #314

wants to merge 2 commits into from

Conversation

kuribas
Copy link
Contributor

@kuribas kuribas commented Aug 5, 2014

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.

@gracjan
Copy link
Contributor

gracjan commented Feb 17, 2015

@kuribas: does this merge replace haskell-indentation with hi2?

In case you did not see results:

In toplevel form:
haskell-indentation.el:1190:1:Error: the function `hl2-show-overlays' is not known to be defined.
make: *** [check-haskell-indentation] Error 1

@gracjan
Copy link
Contributor

gracjan commented Feb 27, 2015

@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.

@kuribas
Copy link
Contributor Author

kuribas commented Feb 27, 2015

It's not an old version of hi2, I merged the changes of hi2 with the current haskell-indentation.el.

@gracjan
Copy link
Contributor

gracjan commented Feb 27, 2015

@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?

@kuribas
Copy link
Contributor Author

kuribas commented Feb 27, 2015

@gracjan: no, I think it's better to remove the hi2 fork, and develop haskell-indentation.el as part of haskell-mode.

@gracjan
Copy link
Contributor

gracjan commented Feb 27, 2015

@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).

@kuribas
Copy link
Contributor Author

kuribas commented Feb 27, 2015

No, haskell-indentation was written from scratch.

@gracjan
Copy link
Contributor

gracjan commented Feb 27, 2015

@kuribas: is there anything provided by haskell-indent that is missing in haskell-indentation (hi2)?

@kuribas
Copy link
Contributor Author

kuribas commented Feb 27, 2015

Yes, the two are totally different modes, with different features and keybindings.

@gracjan
Copy link
Contributor

gracjan commented Feb 27, 2015

@kuribas: that is unfortunate, because haskell-indent does not seem to be maintained.

@kuribas
Copy link
Contributor Author

kuribas commented Feb 27, 2015

I wrote haskell-indentation because I didn't like editing with haskell-indent, so I cannot help you here, sorry.

@gracjan
Copy link
Contributor

gracjan commented Feb 27, 2015

@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.

@hvr
Copy link
Member

hvr commented Feb 27, 2015

@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 equally popular, with a preference on haskell-indent (but still no super-majority).

@gracjan
Copy link
Contributor

gracjan commented Feb 27, 2015

Preference is on haskell-indentation: 43 votes, haskell-indent: 27 votes.

Why hi2 answer is not present in the poll? Wasn't hi2 available back then?

@gracjan
Copy link
Contributor

gracjan commented Feb 27, 2015

@hvr: also note that I've worked a lot on haskell-simple-indent recently so I know how buggy it used to be. Knowing that people prefered haskell-simple-indent to something else is a testimony how many issues did they encounter using other indentation modes.

@kuribas
Copy link
Contributor Author

kuribas commented Feb 27, 2015

@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.

@hvr
Copy link
Member

hvr commented Feb 27, 2015

Yes, hi2 came into existence shortly after that survey...

@gracjan
Copy link
Contributor

gracjan commented Feb 27, 2015

@kuribas: I guess a rebase over master would be needed to get a clean compile.

@purcell purcell closed this Feb 27, 2015
@purcell
Copy link
Member

purcell commented Feb 27, 2015

Came here to close this as suggested, but if I've misunderstood, feel free to reopen.

@gracjan
Copy link
Contributor

gracjan commented Feb 27, 2015

Yes, a bit too fast :)

@purcell
Copy link
Member

purcell commented Feb 27, 2015

Now replaced by #483, I see. Thanks @kuribas

@kuribas
Copy link
Contributor Author

kuribas commented Feb 27, 2015

No problem.

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.

4 participants