Raphael
Raphael
Is the development on this stale? Wondering how I could help out to make it non-alpha.
@marten-de-vries Thanks. I will give it a go and see if I can get the desired behavior implemented. `kneden` is by far the most readable and concise solution re async/await...
I forked this repo and did some tests locally. ``` js // 1. AwaitExpression async function fnAwaitExpression(a){ if(!a) return; await Promise.resolve(a); } // 2. AwaitExpression (transpiled) function _fnAwaitExpression(a) { return...
[A small test case …](https://babeljs.io/repl/#?evaluate=true&lineWrap=false&presets=es2015%2Creact%2Cstage-2&code=%2F%2F%20Util%0Aconst%20log%20%3D%20console.log.bind%28console%29%3B%0A%0A%2F%2F%20AwaitExpression%0Aasync%20function%20fnAwaitExpression%28a%29%7B%0A%20%20if%28!a%29%20return%3B%0A%20%20await%20Promise.resolve%28a%29%3B%0A%7D%0A%0Afunction%20_fnAwaitExpression%28a%29%20%7B%0A%20%20%20%20return%20Promise.resolve%28%29.then%28function%20%28%29%20%7B%0A%20%20%20%20%20%20%20%20if%20%28!!a%29%20%7B%0A%20%20%20%20%20%20%20%20%20%20%20%20return%20Promise.resolve%28a%29%3B%0A%20%20%20%20%20%20%20%20%7D%0A%20%20%20%20%7D%29.then%28function%20%28%29%20%7B%7D%29%3B%0A%7D%0A%0A%2F%2F%20ReturnExpression%20%2B%20AwaitExpression%0Aasync%20function%20fnReturnAwaitExpression%28a%29%7B%0A%20%20if%28!a%29%20return%3B%20%20%0A%20%20return%20await%20Promise.resolve%28a%29%3B%0A%7D%0A%0Afunction%20_fnReturnAwaitExpression%28a%29%20%7B%0A%20%20%20%20return%20Promise.resolve%28%29%0A%20%20%20%20.then%28function%20%28%29%20%7B%0A%20%20%20%20%20%20%20%20if%20%28!!a%29%20%7B%0A%20%20%20%20%20%20%20%20%20%20%20%20return%20Promise.resolve%28a%29%3B%0A%20%20%20%20%20%20%20%20%7D%0A%20%20%20%20%7D%29%0A%20%20%20%20.then%28function%20%28%29%20%7B%7D%29%3B%0A%7D%0A%0Afunction%20_fnReturnAwaitExpressionFIXED%28a%29%20%7B%0A%20%20%20%20return%20Promise.resolve%28%29%0A%20%20%20%20.then%28function%20%28%29%20%7B%0A%20%20%20%20%20%20%20%20if%20%28!!a%29%20%7B%0A%20%20%20%20%20%20%20%20%20%20%20%20return%20Promise.resolve%28a%29%3B%0A%20%20%20%20%20%20%20%20%7D%0A%20%20%20%20%7D%29%3B%0A%7D%0A%0AfnAwaitExpression%28%22a%22%29.then%28v%20%3D%3E%20%7Blog%28v%29%7D%29%3B%20%20%20%2F%2F%3D%3E%20nothing!%0A_fnAwaitExpression%28%22a%22%29.then%28v%20%3D%3E%20%7Blog%28v%29%7D%29%3B%20%20%2F%2F%3D%3E%20nothing!%20Correct%20behavior.%0A%0Alog%28%22------------------%22%29%0A%0AfnReturnAwaitExpression%28%22_b%22%29.then%28v%20%3D%3E%20%7Blog%28v%29%7D%29%3B%20%20%20%2F%2F%3D%3E%20_b%0A_fnReturnAwaitExpression%28%22b%22%29.then%28v%20%3D%3E%20%7Blog%28v%29%7D%29%3B%20%20%2F%2F%3D%3E%20nothing!%20Wrong%20behavior.%20Expected%20b.%0A_fnReturnAwaitExpressionFIXED%28%22_b%22%29.then%28v%20%3D%3E%20%7Blog%28v%29%7D%29%3B%20%20%2F%2F%3D%3E%20_b!%20Correct%20behavior.)
I just did the same test suite against `dev`—works like a charm with incremental build turned on (`"cleanPublic" : true,`). It however breaks with the first error when `"cleanPublic" :...
I investigated this further. It seems to be a bug with the PatternRegistry. On the initial run I get the following dump from it (`pattern_graph.js:131:15`): ``` -------- name: 10-atoms/btn~core.json fileName:...
Maybe @tburny can take a look as well.
This is scheduled for the initial release which should soon happen (almost the same time the Pattern Lab Node core is released to version 3). It is tracked via #49
Add note to readme.md until #49 lands as per #28
@bmuenzenmeyer Is this a viable feature and par with any core functionality?