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

Ref might contain a cleanup function #866

Closed
wants to merge 30 commits into from
Closed

Conversation

davesnx
Copy link
Member

@davesnx davesnx commented Nov 25, 2024

Added another "constructor" for callbackRef to avoid a breaking change, we might add deprecations and further cleanup in the future versions to keep those constructors to the minimum

…esting-libraries

* '19' of github.com:/reasonml/reason-react:
  Add deprecations on ReactDOMTestUtils
  Add uri comment back on action
  Update src/React.re
  Update src/React.re
  Update src/React.re
  Snapshot with lower {}
…on-react into replace-testing-libraries

* 'replace-testing-libraries' of github.com:/reasonml/reason-react:
  Bind React.act and React.actAsync
…on-react into ref-as-valid-prop

* 'replace-testing-libraries' of github.com:/reasonml/reason-react:
  Bind React.act and React.actAsync
  Bind React.act and React.actAsync
  Add deprecations on ReactDOMTestUtils
  Add uri comment back on action
  Add comment on install --force
  Remove jest from demo/dune
  Update src/React.re
  Update src/React.re
  Update src/React.re
  Snapshot with lower {}
…on-react into add-ref-with-cleanup

* 'replace-testing-libraries' of github.com:/reasonml/reason-react:
  Fix hola test :(
  Test react.act and react.actasync
  Use React.act as async version only
@davesnx davesnx closed this Nov 25, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant