add transcript with `edit.namespace` output for `base` and `http`

unisonweb / unison

A friendly programming language from the future

https://unison-lang.org

Other

5.81k stars 271 forks source link

add transcript with `edit.namespace` output for `base` and `http` #5361

Open aryairani opened 2 months ago

aryairani commented 2 months ago

Overview

A place to notice pretty-printing changes.

Implementation notes

Interesting/controversial decisions

It takes a while to run, we may want to split them out into a different test that utilizes some codebase caching.
It produces big output which may clog Github Actions log viewing.
It doesn't try to round-trip them, as they would both fail.
It's pinned to specific releases to avoid spurious output changes

Test coverage

Loose ends

Especially the caching thing.

aryairani commented 1 month ago

It adds to the transcript running time (especially with the clone) but I figure it's better to get it in first, so we can spot how changes affect the output, and speed it up later.

aryairani commented 2 weeks ago

Windows showing a change to the golden transcripts in http output, not sure why yet.