drym-org / qi

An embeddable flow-oriented language.
58 stars 12 forks source link

Correct normalizations #133

Closed countvajhula closed 8 months ago

countvajhula commented 8 months ago

Summary of Changes

Address review by @benknoble on the compiler PR re: the soundness of normalizations.

Public Domain Dedication

benknoble commented 8 months ago

If I have time today I'll confirm this fixes my AoC tests.

countvajhula commented 8 months ago

Thanks for verifying!

countvajhula commented 8 months ago

There were a few compiler tests that were failing for the old rules. I guess this proves that our compiler tests are working 😄