search

presidentbeef / brat

Brat is a little language for people who don't like to be told what to do.
https://www.brat-lang.info/
93 stars 6 forks source link

[core] Fix rescue when throwing Brat string #16

Closed presidentbeef closed 6 years ago

presidentbeef commented 6 years ago

As a result, you can now see the (consistently) failing AST tests from the earlier re-introduction of symbols.

catb0t commented 6 years ago

I think the tests need changing now? 9 failures https://travis-ci.org/presidentbeef/brat/builds/456839497#L1086, it looks like simple string was replacd by simple symbol, or is this related to #17

presidentbeef commented 6 years ago

@catb0t Yes, this fix allowed the actual test failures to be consistently reported. #17 is to fix the test failures.

catb0t commented 6 years ago

Ah, excellent. :+1: