Commit 13449f0
Migrate to modern ReScript .res.js suffix and remove patch
Following Alex Fedoseev's recommendation, migrated from .bs.js to .res.js
suffix, eliminating the need for patching @glennsl/rescript-json-combinators.
Changes:
1. Updated bsconfig.json suffix from .bs.js to .res.js
2. Removed patch for @glennsl/rescript-json-combinators
3. Updated webpack config to resolve .res.js extensions
4. Rebuilt ReScript - all files now generate with .res.js suffix
5. Removed duplicate RescriptShow/ror_components directory
6. Updated patches/README.md to document migration
Why this is better:
- ✅ No patch maintenance burden
- ✅ Follows modern ReScript v11+ conventions (.bs being phased out)
- ✅ Compatible with future ReScript versions
- ✅ Works with all ReScript packages out of the box
Upstream discussion: glennsl/rescript-json-combinators#9
🤖 Generated with [Claude Code](https://claude.com/claude-code)
Co-Authored-By: Claude <noreply@anthropic.com>1 parent 4d9d19e commit 13449f0
File tree
18 files changed
+771
-71
lines changed- client/app/bundles/comments/rescript
- Actions
- CommentForm
- forms
- CommentList
- AlertError
- Comment
- Header
- bindings
- config/webpack
- patches
18 files changed
+771
-71
lines changed| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
13 | 13 | | |
14 | 14 | | |
15 | 15 | | |
16 | | - | |
| 16 | + | |
17 | 17 | | |
18 | 18 | | |
19 | 19 | | |
| |||
Lines changed: 62 additions & 0 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.
Lines changed: 215 additions & 0 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.
Lines changed: 77 additions & 0 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.
0 commit comments