Skip to content

Conversation

@mununki
Copy link
Member

@mununki mununki commented Jan 8, 2024

Regarding the doc changes, rescript-lang/rescript-lang.org#777 (comment)

This PR changes the automatic mode of JSX mode to default.

@mununki mununki requested a review from cknitt January 8, 2024 16:30
@mununki
Copy link
Member Author

mununki commented Jan 8, 2024

I bump to the test failure about dead code: https://github.com/rescript-lang/rescript-compiler/actions/runs/7450332135/job/20269089490?pr=6552 I don't think this is caused by this PR changes.

Therefore I added [@@live] to it for passing the test. d32a129

@cknitt
Copy link
Member

cknitt commented Jan 8, 2024

@mununki Thanks!

The PR contains some unrelated changes to jscomp/others/belt_HashSetInt.resi and jscomp/others/belt_HashSetString.resi though - could you remove those?

@mununki
Copy link
Member Author

mununki commented Jan 9, 2024

@mununki Thanks!

The PR contains some unrelated changes to jscomp/others/belt_HashSetInt.resi and jscomp/others/belt_HashSetString.resi though - could you remove those?

Sure. I don't know why these changes are made after running tests in my local.

@mununki
Copy link
Member Author

mununki commented Jan 9, 2024

@cknitt cknitt merged commit 58f5d36 into master Jan 9, 2024
@cknitt cknitt deleted the jsx-automatic-default branch January 9, 2024 07:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

3 participants