Renée

Results 469 comments of Renée

Had to take some time to get used to this massive refactor, but I like the `createElement` change for sure :+1: appreciate your work on this!

@mjstahl yeah, hyperx is not an easy codebase. i'd have to go in with some console.logs myself to be sure, but I think the places to look at would be...

Where is it happening? We don't use estraverse directly.

A way to do this today is using object spread: ```js const optional = {} if (nullableValue) optional['data-something-optional'] = nullableValue html` ` ``` it's a bit verbose but I think...

it looks like it's missing from the docs, yes. it is definitely a tested and supported feature, though!

@serapath you might be interested in #102 if you didn't see it yet!

Woah that is super neat! Thanks for sharing :D

Thanks for the report! This is a hyperx bug, probably the same as https://github.com/choojs/hyperx/issues/34 or https://github.com/choojs/hyperx/issues/41. Hard to say how hard the fix would be.

> _Or_ add an `export = ` line to make `import bel = require('bel')` work? This sound like the best approach to me.

Sounds like a bug imo!