Do not call ocamlformat from dune

[georgeee]

It's a trick that doesn't seem to be necessary and isn't the cleanest one.

Explain how you tested your changes:

• Mina compiles

Checklist:

• Dependency versions are unchanged
  • Notify Velocity team if dependencies must change in CI
• Modified the current draft of release notes with details on what is completed or incomplete within this project
• Document code purpose, how to use it
  • Mention expected invariants, implicit constraints
• Tests were added for the new behavior
  • Document test purpose, significance of failures
  • Test names should reflect their purpose
• All tests pass (CI will check this if you didn't)
• Serialized types are in stable-versioned modules
• Does this close issues? None

[mrmr1993]

Recommend closing.

[mrmr1993]

We want to run ocamlformat here, otherwise the results are extremely ugly. The crypto team needs to explicitly look at these files.

[georgeee]

I see...

I assumed formatting was introduced mainly to avoid issues with committing files and running make reformat-diff before commit.

[dannywillems]

There is a more elegant solution which is using built-in fmt in dune, with dune fmt, and get rid of reformat. Source: https://ocaml.org/docs/formatting-your-code