Skip to content

Fix bisect ppx - #6

Open
jchavarri wants to merge 8 commits into
mainfrom
fix-bisect-ppx
Open

jchavarri wants to merge 8 commits into
mainfrom
fix-bisect-ppx

Conversation

@jchavarri

Copy link
Copy Markdown
Member

No description provided.

@anmonteiro

Copy link
Copy Markdown
Member

Tests say:

  • needs melange-webapi for tests
  • Unbound module Bisect:

File "react/src/ReactTestingLibrary.re.ml", line 1:
Error: Unbound module Bisect

Not sure if this is ready for review but just letting you know.

@jchavarri

Copy link
Copy Markdown
Member Author

@anmonteiro thanks for the heads up. took a while, but it seems now everything's good.

Comment thread jest.config.js Outdated
Comment on lines +4 to +6
// ignoring these, are they are just temporary copies from runtime_deps, the final ones are in
// the path with melange.emit target, e.g. _build/default/dom/test/test/dom/test/__snapshots__
modulePathIgnorePatterns: ["react/test/__snapshots__", "dom/test/__snapshots__"]

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

trying my best 😄

@anmonteiro anmonteiro left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

looks like a painful setup tbh

@jchavarri

Copy link
Copy Markdown
Member Author

looks like a painful setup tbh

the bisect or the jest one?

@anmonteiro

Copy link
Copy Markdown
Member

The Bisect setup

@davesnx

davesnx commented Mar 12, 2026

Copy link
Copy Markdown
Member

I removed bisect_ppx for now, until it works with latest ppxlib

4051026

41a38b0

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.

3 participants