Sophie Alpert

Results 99 comments of Sophie Alpert

(We added this to the parser originally in https://github.com/facebookarchive/esprima/pull/9 but never made the transformer (jstransform at the time) support it.)

@RReverser Hmm? I didn't mention Babel.

@ThomWright That .call shouldn't cause the warning. Not sure off the top of my head what causes it; probably something in art internals.

If you look at the implementation of that warning in the React repo you'll see that it intercepts .bind calls, not .call. It's mostly meant as a warning to prevent...

> So, when you `.bind(null, ...)`, the `event` parameter is undefined. That doesn't sound right. The first arg is used as context (i.e., `this`), not as the first arg of...

By the way: require() with a dynamic argument doesn't work in many bundlers, including Facebook's. We also don't want to send down code that we won't ever run. Perhaps we...

At the very least, you should be using the production (minified, NODE_ENV=production) version of React.

Sounds accurate to me. Fastly can probably also do it; their config options are more flexible (at least for the plan we have).

I’m not aware of any cases where the exponential distribution is subpar (including as compared to a constant interval), so I’d advocate for it as the default (likely only) option....

> Or you can also start profiling instances with the next interval That's fair but looks inefficient and isn't close to a "pit of success". In my mind the point...