Conversation
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
- Eliom_service -> Eliom.Service, etc. - Os_session -> Os.Session, etc. - Fix icons with named ICON_HTML signature - Use Eliom.Eliom_common for scopes (not yet renamed) Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
…rule Eliom_react was not renamed (conflict with react package). The sed pipe for deriving json is no longer needed since ocsigen-i18n now generates [@@deriving json] directly. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Replace the backward-compatible wrapper command with the canonical ocsigen-i18n --eliom invocation.
Ocsigen_extensions -> Ocsigen.Extensions Ocsigen_config -> Ocsigen.Config Ocsigen_server -> Ocsigen.Server Replace client-side Ocsigen_config shim with local debugmode function. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Replaces the local ad-hoc ICON_HTML signature workaround with the shared signature now exposed by Eliom for [F] and [D]. Same goal (avoid the [module type of Eliom.Content.Html.F] strengthening bug in wrapped libraries), but with no local boilerplate. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Tracking PR for the modernized line of the template: wrapped module names (Eliom.Xxx, Os.Xxx, Ot.Xxx, Ocsigen.Server), ocsigen-i18n --eliom, Eliom.App.run, etc.
This branch corresponds to the modernize/deriving branches of eliom, ocsigen-start, ocsigen-toolkit and ocsigenserver. Do not merge until that stack is released; master stays on the non-modernized line matching the master branches.
Note: master received the client PPX order fix (#8) after the split; merging master into this branch (or cherry-picking) will bring it here. The deriving work on top of this line is tracked separately in #5.