grain icon indicating copy to clipboard operation
grain copied to clipboard

chore(compiler): Properly handle disable_stamps state

Open ospencer opened this issue 9 months ago • 1 comments

I noticed that we weren't properly resetting the disable_stamps state in a handful of tests, causing a good chunk of the test suite to not have stamps in their snapshots. For some test suites, this would cause them to fail if run individually.

There's somewhat of a discussion to be had for disabling stamps in a bunch of the snapshots, but I find the stamps really helpful for debugging and think we should keep them. (They really only change when we add new global types.)

ospencer avatar Apr 12 '25 03:04 ospencer

Closes: #2235

spotandjake avatar Apr 12 '25 04:04 spotandjake