fix: bind scope to null to prevent issues with inline requires - Githubissues

software-mansion / react-native-gesture-handler

Declarative API exposing platform native touch and gesture system to React Native.

https://docs.swmansion.com/react-native-gesture-handler/

MIT License

6.13k stars 982 forks source link

fix: bind scope to null to prevent issues with inline requires #2972

Closed EvanBacon closed 4 months ago

EvanBacon commented 4 months ago

Description

I've identified an issue with Metro's inlineRequires + experimentalImportSupport while building tree shaking support for Expo CLI. This issue causes an invalid invocation to be thrown because the scope of the build-in functions is not correct.
More info can be found here https://github.com/EvanBacon/inline-requires-issue

Test plan

In my repro the bindings no longer throw. I've raised the issue with Metro team so we can create a system-wide fix, but this should prevent any issues in the meantime.