typelevel / weaver-test

A test framework that runs everything in parallel.
http://typelevel.org/weaver-test/
Other
30 stars 8 forks source link

Use munit diff logic #25

Closed zainab-ali closed 2 months ago

zainab-ali commented 2 months ago

As part of #15 , we wish to improve the diff rendering of the Comparison trait by using a Myers diff implementation. munit's Myers diff algorithm is a prime candidate.

As discussed on Discord, we prefer to vendor the diff logic than pull in an external dependency.

This PR is an experiment in vendoring.

Note that:

zainab-ali commented 2 months ago

To summarize the discussions on this, there are two approaches we could take to improving the diff logic:

I'm exploring pulling in the library dependency, but shading it such that users of weaver and munit are unaffected. Given that the lib is unlikely to change, but is on the same release cadence as munit, releases will also be excluded from Scala Steward.

Baccata commented 2 months ago

Changes look good 👍

Do you have a rough idea of when munit-diff could be released ?

zainab-ali commented 2 months ago

Thanks! The PR has no obvious outstanding issues, so I expect it to be merged in the next few days. I'm not sure of the munit release cycle - @tgodzik may be able to comment.

tgodzik commented 2 months ago

Thanks! The PR has no obvious outstanding issues, so I expect it to be merged in the next few days. I'm not sure of the munit release cycle - @tgodzik may be able to comment.

We plan to do munit 1.0.0-RC1 after that is merged and if nothing pops up later on, we'll release a full 1.0.0

tgodzik commented 2 months ago

Munit and munit-diff 1.0.0-RC1 is out. I also released manually for 0.4 Scala Native. Do let me know if it works!

Baccata commented 2 months ago

Merging even if it depends on an RC, because it's shaded anyway, and because there's still a little bit of work to be done before an actual release of weaver, which may give munit enough time for a 1.0.0 release