alexschneider / teascript

5 stars 2 forks source link

Add support for conditionals #52

Closed alexschneider closed 9 years ago

alexschneider commented 9 years ago

Right now, this allows for the following:

if true: out("false")
else if false:
  out("true")
else: out("maybe")

because support for that is trivial to add (unlike we feared). If anyone doesn't like that, we can easily only only allow blocks for combined if, else if, else expressions, and only allow the single line for if expressions alone.

alexschneider commented 9 years ago

To save headaches in the future, if people install new modules, the npm-shrinkwrap.json needs to be updated!

alexschneider commented 9 years ago

@rachelriv Updated, please merge

rachelriv commented 9 years ago

@alexschneider LGTM but can you please rebase so it can be merged automatically?

alexschneider commented 9 years ago

@rachelriv done

rachelriv commented 9 years ago

It looks like one of your parser tests if failing. This should be good once all tests are passing.

rachelriv commented 9 years ago

LGTM.