fix: target es2018

[thebuilder]

Closes #635

[vercel]

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name	Status	Preview	Updated (UTC)
react-intersection-observer	✅ Ready (Inspect)	Visit Preview	Jun 28, 2023 10:43am

[codesandbox-ci]

This pull request is automatically built and testable in CodeSandbox.

To see build info of the built libraries, click here or the icon next to each commit SHA.

Latest deployment of this branch, based on commit 4c110ac:

Sandbox	Source
useInView	Configuration
InView render props	Configuration
InView plain children	Configuration

[alexsorokoletov]

Thank you for this, @thebuilder!

Wanted to share a bit of feedback - in our legacy app with webpack 3 (yeah, I know), the versions 9.5.x including 9.5.2 are still failing because of the spread operator, so we had to pin 9.4.x instead.

Upgrading webpack side of the build is not possible in our project at this moment (otherwise we would've done that).

Thanks for keeping this package up to date and resolving all the issues, and thank you for this OSS package!

[thebuilder]

Thank you for this, @thebuilder!

Wanted to share a bit of feedback - in our legacy app with webpack 3 (yeah, I know), the versions 9.5.x including 9.5.2 are still failing because of the spread operator, so we had to pin 9.4.x instead.

Upgrading webpack side of the build is not possible in our project at this moment (otherwise we would've done that).

Thanks for keeping this package up to date and resolving all the issues, and thank you for this OSS package!

You might be able to get around it, by explicitly including packages in your Webpack rules, so they'll be transformed by Babel.

[alexsorokoletov]

Thank you @thebuilder, that might be a valid path forward.
There are few layers of infrastructure above the webpack (main reason why it can't be migrated easily) and I wanted the information for anyone else with a similar lock in and a problem