Repro case for loading too many rules for @fmt - #16364
Leonidas-from-XIV wants to merge 1 commit into
Conversation
Signed-off-by: Marek Kubica <marek@tarides.com>
|
Maybe what's needed here is some sort of |
|
Maybe. I was also wondering whether it would make sense to coerce evaluation failures in |
|
The issue with the cram stanza is that we need to interpret the enabled_if as early as possible. It can potentially contribute to the I'll note that I haven't thought through what |
When doing
dune build @fmtDune loads the rules and, given the package does not exist fails to evaluate the pform. However in the case ofcramstanzas, there's no need to evaluate these pforms as the result should not matter for formatting.Originally reported by @jonludlam in ocaml/odoc#1490