Raphael

Results 14 comments of 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:...

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?