Rene Saarsoo

@jimmywarting The first bug relating to `super()` is already known: #186 Regarding this `instanceof` check. When some code relies on this, then converting this function to class was really a...

Nope. It's not extensible. I suggest you instead look into tools like jscodeshift or babel-codemod, which provide the ease of extensibility. E, 22. oktoober 2018 23:22 Jimmy Wärting kirjutas: >...

Yeah. Actually the current implementation tries to ignore the redeclared variables, but the result is not pretty. Probably better to convert the redeclares to simple assignments. This should actually simplify...

The tricky case in here is this scenario: ```js function foo() { if (true) { var x = 1; } else { var x = 2; } return x; }...

One should also consider a very similar case: ```js function foo() { if (true) { var x = 1; return x; } else { var x = 2; return x;...

So, I switched to node 12.22.1 and tried to run the tests again: ``` rm -rf node_modules npm ci npm init npm run build npm run test ``` I still...

> Do you need the Vue tests to run? Well, I was trying to make some changes across the whole repo... but when I'm not starting out with green tests,...

SQL Formatter does have placeholder support, however it's designed to work with value placeholders as in prepared statements. These integrate well with SQL, as in many SQL dialects the placeholder...

Yes. Quite unlikely. The parsing of SQL is complex enough without adding templating syntax which would make the task theoretically impossible.

Actually... turns out that the formatter actually supports `${name}` placeholders. I had completely forgotten that we had support for these for Hive and Spark languages. (I thought we had dropped...