Hrusikesh Panda
Hrusikesh Panda
> That could also avoid the problem of semver ranges Or you could specify the range as well, kinda like browserlist config.
+1 for debug only support. Combine/minification etc. (if there is popular demand) can be rolled out in a later version. Most Production deployments do not serve source maps anyway (security,...
You can have the PR in draft state and exclude it from autoupdate via `PR_READY_STATE: ready_for_review`, no?
Based on my real world experience, here are my thoughts: Children props approach is the cleanest in terms of the component API - but it's also the quirkiest because children...
> I think one downside of the HoC is that MyVirtualList has to be recreated every render Not unless you're creating the HoC inside render or somewhere that leads to...
> This will work for singular uses of the component, but having multiple instances of a component that use VirtualList() would cause problems. Maybe I'm not picturing this right. What...
@developerdizzle I like the sound of it. Do you have a branch or any sample code to look at?
I know this is WIP but couldn't help notice these things. Apologies if some of these are things you're already working on or this is not the type of feedback...
I think 2 things will be a roadblock: - [Raven reachability](https://github.com/ewolfe/prlint/blob/master/src/app.js#L13) - Internally hosted repos may not like the idea of reaching out to a public endpoint with their info...
And I'll love to pair on this! Let me know if you want me to send a PR for above.