Use import for lodash/flowRight to allow for tree shaking

[Pajn]

browser.ts exports lodash/flowRight as compose, however it import this module using require()which makes it impossible for Webpack to treeshake it away. This causes quite a bit of lodash to be pulled in even though compose from react-apollo is not used at all. Replacing this require call with import should allow it to be treeshaked and its cost only paid if actually used.

react-apollo/src/browser.ts

Line 27 in 9d02df8

const compose = require('lodash/flowRight');

[rosskevin]

You could PR this just like #2045. At least it would solve the problem in the next patch/minor release before it is removed entirely.

[Pajn]

The thing is that I have no idea if changing it to

import * as compose from 'lodash/flowRight';
export { compose };

would break stuff for some users.

I have not used require imports since Typescript 1.6 is released so there might very well be minor differences that I have forgotten.
Since you use ES imports everywhere else they question is rather, why isn't this one? Is there a reason?

[Pajn]

Okey, types from DefinitlyTyped where bad. That I guess that was simply the reason. See the PR