[Req] Flag to ignore errors

[pakoito]

I currently have a codebase that mixes classes and regular files that I'd like formatted in one go. At the moment, all the errors when Fatal error: exception ("Internal Error: classes not implemented") create an empty file that's not cleaned up and pollutes the repository.

I'd like to have an option --ignore-errors that skips files with errors, or just logs them to console.

[hcarty]

One option you could use for now is to add a [@@@ocamlformat.disable] annotation to the source files which don't format properly. This will cause ocamlformat to leave their code unmodified.

Have you tried the latest masterbranch of ocamlformat? There have been several class- and object-related fixes which may help as well.

[pakoito]

I'll check with the other owners re annotations.

I have not tried master. Is it possible to pin the latest master from opam? Or does it require to build from source? I'm fairly new to the ecosystem :D

[hcarty]

opam pin add ocamlformat --dev-repo should do it. That will give you a build from the latest commit to master.

[pakoito]

Thanks! What's the command to pin it back to a release version?

[hcarty]

opam pin remove ocamlformat

[pakoito]

Perfect. Thanks again ^^

[pakoito]

Should I close this ticket, or is it a suggestion worth considering?

[pakoito]

Ping

[hhugo]

Keep it open. I'd love to have this as well. I might even have a patch for it already.

[hhugo]

I think we can close this one now that #243 is merged:

• process all files instead of stopping at the first error.
• accept a --quiet flag to quiet errors due to invalid source files.

[pakoito]

Thank you @hhugo! That's two coffees I owe you this week ^^