Renée
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!