search

tweag / topiary

https://topiary.tweag.io/
MIT License
580 stars 29 forks source link

CLI Changes Sub-PR: Formatting subcommand #599

Closed Xophmeister closed 1 year ago

Xophmeister commented 1 year ago

This forms a part of PR #583 and must not be merged into main alone. It is only the implementation of isolated functionality -- and thus not expected to necessarily pass the CI checks -- to split up the work into logical chunks.

Xophmeister commented 1 year ago

@ErinvanderVeen It turns out that cache was working, but the locking is suboptimal. I've so far been unable to fix it, so I've just left it in there, with a FIXME note. However, fixing the cache shouldn't change main.rs, so once this PR is merged into chris/cli-changes, you can rebase from that to prevent being blocked, while I work on the tests.