Skip to content
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

ocp-indent #2

Open
eriklovlie opened this issue Feb 9, 2016 · 2 comments
Open

ocp-indent #2

eriklovlie opened this issue Feb 9, 2016 · 2 comments

Comments

@eriklovlie
Copy link
Owner

Seems to me it would make a lot of sense to use ocp-indent to provide standard indentation. I'm not at all sure how language-ocaml works, but maybe that can be forked to only do highlighting and then another package could use "ocp-indent --lines - --numeric". This could be done on the fly (on newlines I suppose) and only for the lines that have changed.

Could probably follow the approach from the emacs package: https://github.com/OCamlPro/ocp-indent/blob/master/tools/ocp-indent.el

Technically it would be nicest if this was another package. Doesn't seem to be any point in bundling this in linter-ocaml. But I'll record the idea here for now.

It would be incompatible with language-ocaml however, so would need to also provide highlighting (and somehow mark itself as incompatible with language-ocaml so the user doesn't get weird behavior).

@eriklovlie
Copy link
Owner Author

I'm still confused about how atom language packages specify indentation rules. E.g. there following query returns nothing: https://github.com/toroidal-code/language-ocaml/search?utf8=%E2%9C%93&q=increaseIndentPattern

@Khady
Copy link

Khady commented Apr 11, 2016

There is now a package to use ocp-indent in atom: https://atom.io/packages/ocaml-indent

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants