feat: add Compare::max and Compare::min

[illusory0x0]

No description provided.

[coveralls]

Pull Request Test Coverage Report for Build 5708

Details

• 0 of 6 (0.0%) changed or added relevant lines in 1 file are covered.
• No unchanged relevant lines lost coverage.
• Overall coverage decreased (-0.09%) to 92.569%

---

Changes Missing Coverage	Covered Lines	Changed/Added Lines	%
builtin/traits.mbt	0	6	0.0%

Totals
Change from base Build 5706:	-0.09%
Covered Lines:	6029
Relevant Lines:	6513

---

💛 - Coveralls

[bobzhang]

@illusory0x0 we have a high bar when adding new traits or modify existing traits, can you elaborate why it is needed, does it justify?

[illusory0x0]

@illusory0x0 we have a high bar when adding new traits or modify existing traits, can you elaborate why it is needed, does it justify?

test {
  let x = 132
  let y = 2323
  let z = 2323 
  String::max(x, y) |> ignore // use monomorphic functions would help type infer
  (@math.maximum : (Int,Int) -> Int)(x,y) |> ignore

  x.max(y) |> ignore // infix syntax
  @math.maximum(x,y) |> ignore // prefix syntax

  x.max(y).max(x) |> ignore // easy to writing and avoid nesting
  @math.maximum(x,@math.maximum(y,z)) |> ignore
}