phillipstanleymarbell / Noisy-lang-compiler

Noisy language compiler
MIT License
17 stars 1 forks source link

Issue 637 improve test framework #638

Closed PeiMu closed 1 year ago

PeiMu commented 1 year ago

address #637

PeiMu commented 1 year ago

The test framework would be better base on the newest version of the compiler, so I merged #634 here.

PeiMu commented 1 year ago

Could you please review and merge this PR? @KomaGR @blackgeorge-boom

PeiMu commented 1 year ago

Easily fix Issue #639 here.

PeiMu commented 1 year ago

Please see my comments on Slack.

done. please check and merge it.

PeiMu commented 1 year ago

Thank you for the changes. I would like to ask you to please address the further comments I have made.

Done. Can you check it again? Thanks.

PeiMu commented 1 year ago

Just a reminder (mainly for myself), this PR won't be changed anymore. Bug fixes or new features will add in #643 or new issues.