Pattern-matching of `bool` value is formatted to ternary if we match `true` first

[denis-ok]

When pattern-matching boolean values, it's common to match the true value first. However, this approach can lead to undesirable formatting behavior. For instance:

This:

switch (condition) {
| true => ()
| false => ()
};

Formats to:

condition ? () : ();

Sometimes a code style when we first match a shorter branch (with less code) is preferred. To prevent automatic formatting to a ternary expression in such cases we have to match false as a wildcard _false.

switch (condition) {
| true =>
  // short code
  ()
| _false => 
  // long code
  // long code
  // long code
  ()
};

[anmonteiro]

This has been considered a feature of Reason since the start, but personally it's always annoyed me.

I'm sure some folks depend on it, though, so not sure what to do about it.

[anmonteiro]

there's some discussion in https://github.com/reasonml/reason/issues/1700

[anmonteiro]

let's follow this in #1700