Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: undo the clear observers in test-utils #523

Merged
merged 1 commit into from
Nov 25, 2021

Conversation

thebuilder
Copy link
Owner

#519 and #522 had the side effect of including all the source files in the package, because the test-utils.ts was importing the files. The package should only include the bundle (cjs,esm,umd) files, and the test-utils.js file.

Unverified

This commit is not signed, but one or more authors requires that any commit attributed to them is signed.
@vercel
Copy link

vercel bot commented Nov 25, 2021

This pull request is being automatically deployed with Vercel (learn more).
To see the status of your deployment, click below or on the icon next to each commit.

🔍 Inspect: https://vercel.com/the-builder/react-intersection-observer/9ScyNC3xPShNMPAK8QUnt1vXx3rM
✅ Preview: https://react-intersection-observer-git-revert-519-c-f965ec-the-builder.vercel.app

@codesandbox-ci
Copy link

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 31887cc:

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

@thebuilder thebuilder merged commit 249de1a into master Nov 25, 2021
@thebuilder thebuilder deleted the revert/519-clear-observers branch November 25, 2021 12:26
@thebuilder
Copy link
Owner Author

🎉 This PR is included in version 8.32.5 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant