Anish Aggarwal comments

Results 18 comments of


                                            Anish Aggarwal

Allow props to be set dynamically through knobs on multiple components

But we can add like child component and some other set of props for it, then it can be realistic.

Allow variables inside the function body

``` ()=>{ const on; // talkingAboutThisVariable return } ```

Allow variables inside the function body

yes but how to get that ` const label = 'ok'; ` within the code body ?

Stars icon has larger clickable area

Its the label area . I don't think it's an issue

Remove delete_after_merge functionality

sure, @asvoboda I have removed it from some tests too and even from the `Readme`

Remove delete_after_merge functionality

Oh, I think that after changing the `test config` I need to make changes to the expected output also. Am I correct?

Remove delete_after_merge functionality

> It looks like the head var L158 in merge.go is now unused, and the compiler is complaining. Yeah, replaced it with an `_`.

Remove delete_after_merge functionality

@asvoboda can we merge this one?

Remove delete_after_merge option

Hi @asvoboda, I would like to work upon this issue if there's no one assigned to this one

ByteBufferView: implement most Collection/Sequence customisation points

> @trungducc thank you, that's awesome. I think this is just a misunderstanding. I think we need to compare that the `bounds` parameter of `_failEarlyRangeCheck` lies within `self._range`'s bounds (`self._range.lowerBound`...