Skip to content

V12 migration guide #1048

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.

Sign up for GitHub

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

Jump to bottom
Open
wants to merge 15 commits into
base: master
Choose a base branch
from
Open

V12 migration guide #1048

wants to merge 15 commits into from

Conversation

@fhammerschmidt
Copy link
Member

@fhammerschmidt fhammerschmidt commented Jun 12, 2025 •
edited
Loading

Fixes #945

@cloudflare-workers-and-pages
Copy link

cloudflare-workers-and-pages bot commented Jun 12, 2025 •
edited
Loading

Deploying rescript-lang-org with  Cloudflare Pages  Cloudflare Pages

Latest commit: b6991eb
Status: ✅  Deploy successful!
Preview URL: https://e281c3e1.rescript-lang.pages.dev
Branch Preview URL: https://v12-migration-guide.rescript-lang.pages.dev

View logs

@fhammerschmidt
Copy link
Member Author

fhammerschmidt commented Jul 14, 2025 •
edited
Loading

rescript build -w is deprecated
We need to mention it's now rescript watch

mediremi
mediremi reviewed Jul 24, 2025
View reviewed changes
@nojaf
Copy link
Member

nojaf commented Sep 6, 2025

Somewhere in this guide, we should mention that people need to bump to latest @rescript/react as it still uses old pipe (|>) syntax.

@fhammerschmidt
Copy link
Member Author

fhammerschmidt commented Nov 6, 2025

Concerning pipe-last:
V11 formats it away, but only in uncurried mode. Which is another prerequisite for v12 anyways.

@nojaf
Copy link
Member

nojaf commented Nov 11, 2025

@fhammerschmidt sorry for pushing to this branch, I hope I didn't mess anything local up on your end.
I've added some stuff I think we need to include.
No strong opinion on the exact format, adding the info was my main goal.

tsnobip
tsnobip reviewed Nov 11, 2025
View reviewed changes
@fhammerschmidt
Copy link
Member Author

fhammerschmidt commented Nov 11, 2025

It's fine. In fact we could and should crowdsource more articles.

@fhammerschmidt fhammerschmidt changed the title WIP: V12 migration guide V12 migration guide Nov 13, 2025
@rescript-lang rescript-lang deleted a comment from vercel bot Nov 18, 2025
cknitt
cknitt requested changes Nov 18, 2025
View reviewed changes
nojaf
nojaf reviewed Nov 19, 2025
View reviewed changes
Copy link
Member

@nojaf nojaf left a comment

Choose a reason for hiding this comment

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

Please use more suggestions

@fhammerschmidt fhammerschmidt force-pushed the v12-migration-guide branch 2 times, most recently from 78eb2f8 to 30a90ce Compare November 21, 2025 19:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@tsnobip tsnobip tsnobip left review comments

@nojaf nojaf nojaf left review comments

@mediremi mediremi mediremi left review comments

@shulhi shulhi shulhi approved these changes

@cknitt cknitt Awaiting requested review from cknitt

Requested changes must be addressed to merge this pull request.

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Migration Guide

7 participants

@fhammerschmidt @nojaf @cknitt @shulhi @tsnobip @mediremi