Chore: Remove remnants of Flow

[sapegin]

Remove pieces of Flow left after the TypeScript migration.

[lgtm-com]

This pull request fixes 1 alert when merging 20a8dd2 into 847f917 - view on LGTM.com

fixed alerts:

• 1 for Unused variable, import, function or class

[elevatebart]

I believe we have to keep @babel/preset-flow for some tests.
Apart from that and the small alert, I see no pb with this one.

[lgtm-com]

This pull request fixes 1 alert when merging 8285bd5 into 3bd566b - view on LGTM.com

fixed alerts:

• 1 for Unused variable, import, function or class

[elevatebart]

I have added two changes, please check them out

• add back flow preset for Props.spec.tsx
• make index.ts export a default object to avoid mistakes (potential breaking change)

[lgtm-com]

This pull request fixes 1 alert when merging 283b329 into 3bd566b - view on LGTM.com

fixed alerts:

• 1 for Unused variable, import, function or class

[codecov]

Codecov Report

Merging #1549 into master will not change coverage by %.
The diff coverage is 100.00%.

Impacted Files	Coverage Δ
babel.config.js	100.00% <ø> (ø)
src/loaders/utils/client/evalInContext.ts	100.00% <ø> (ø)
src/loaders/utils/client/requireInRuntime.ts	100.00% <ø> (ø)
src/scripts/index.esm.ts	100.00% <100.00%> (ø)

[sapegin]

We'll need to updates the docs then: https://react-styleguidist.js.org/docs/api.html

[elevatebart]

I agree, I will take care of it before tomorrow. Bart Ledoux Join me on LinkedIn Or as @elevatebart on Discord, Twitter and Github

…

On Mar 8, 2020, 16:08 -0500, Artem Sapegin ***@***.***>, wrote: We'll need to updates the docs then: https://react-styleguidist.js.org/docs/api.html — You are receiving this because your review was requested. Reply to this email directly, view it on GitHub, or unsubscribe.

[elevatebart]

I updated the documentation using imports instead of requires. I hope you find it useful.
It might be useful to add a note about NodeJs not accepting es6 imports. There is a non obvious need for transpilation.

[lgtm-com]

This pull request fixes 1 alert when merging 0dd7c84 into 106df96 - view on LGTM.com

fixed alerts:

• 1 for Unused variable, import, function or class

[sapegin]

I think we can also reexport it as CommonJS in a separate module to avoid breaking changes.

[elevatebart]

I did my best. I hope you like it.

[lgtm-com]

This pull request fixes 1 alert when merging 59e97ef into 106df96 - view on LGTM.com

fixed alerts:

• 1 for Unused variable, import, function or class

[sapegin]

I'm not sure the module field is useful here, I think it's only used by bundlers, but good enough for now ;-) Thanks for the changes!

[styleguidist-bot]

🎉 This PR is included in version 11.0.1 🎉

The release is available on:

• npm package (@latest dist-tag)
• GitHub release

Your semantic-release bot 📦🚀