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

Format reason codebase with ocamlformat? #2676

Open
davesnx opened this issue Nov 12, 2022 · 1 comment
Open

Format reason codebase with ocamlformat? #2676

davesnx opened this issue Nov 12, 2022 · 1 comment

Comments

@davesnx
Copy link
Member

davesnx commented Nov 12, 2022

There’s a bit of formatting issues on a few places on the codebase that could benefit a lot from being fixed.

Do we consider an option to adopt ocamlformat and which are the minimum set of options that allow less changes on the codebase?

PS: If we ever do that, we could add an ignored from history commit so it won’t pollute the history (for the ones that care).

@anmonteiro
Copy link
Member

I'm in favor of this. I've been using OCamlformat in every project and I like not having to think about indentation. PR welcome.

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

No branches or pull requests

2 participants