Denis Bardadym

Results 80 comments of Denis Bardadym

There is a CLI tool for this https://github.com/btd/rollup-plugin-visualizer#cli already. Not sure if i want to support merge as API

Basically merge happen there https://github.com/btd/rollup-plugin-visualizer/blob/master/bin/cli.ts#L70C3-L102C5 I can extract this to the function, if it works for you

The failing tests is exactly where is bug happen. Wrong 'close' event semantic.

@mcollina done. But have a question: according to [docs](https://nodejs.org/dist/latest-v10.x/docs/api/stream.html#stream_event_close_1), close event is not required to be emitted. So now it is a bit unclear how this stream should handle it.

Ping @etaty .This PR very useful, i hit exactly the same limitation.

Just my 5 cents. I have tried to prepend all commands with `application:` prefix. And it shows menu as it should: ![2015-06-06 2 44 42](https://cloud.githubusercontent.com/assets/334851/8017106/402544e0-0bf6-11e5-8493-b2cee1ba9355.png) But pressing to any item...

Hi, sorry. I will not be able to add this anytime soon. But i am very open to either TS migration or addition of types if you mind.

Just create .d.ts file (one or two - depending if you want it to be similar to how tsc generate it). I will increment version when will do publish

Hi, this parameters mostly was copied from original implementation. I made it sure that results of this lib match to libphash. In theory parameters could improve hash, but it very...

Yea, i was thinking about this for some time.