catbird icon indicating copy to clipboard operation
catbird copied to clipboard

Setup MiMa for finagle

Open armanbilge opened this issue 3 years ago • 2 comments

Give it a try with rootFinagle/mimaReportBinaryIssues :)

armanbilge avatar Feb 02 '22 02:02 armanbilge

Codecov Report

Merging #430 (976e505) into main (e1b9fc2) will not change coverage. The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #430   +/-   ##
=======================================
  Coverage   90.32%   90.32%           
=======================================
  Files           7        7           
  Lines         124      124           
  Branches        2        2           
=======================================
  Hits          112      112           
  Misses         12       12           

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

codecov-commenter avatar Feb 02 '22 02:02 codecov-commenter

This is pretty cool. Looking forward to trying it out for real when they update upstream—hopefully it will be able to tell us whether whatever version is next (I was expecting 22.5.0, but you'd think that would be been released by now) is compatible with 22.4.0.

bpholt avatar May 31 '22 21:05 bpholt

I tried setting val versions = Seq("22.7.0", "22.4.0") in FinaglePlugin. Running rootFinagle/mimaReportBinaryIssues reported 89 issues, which indicates that v22.7.0 is not bin-compatible with v22.4.0. (Looking at the "Breaking API Changes " in its release notes, I think that's right!)

bpholt avatar Oct 20 '22 04:10 bpholt