Skip to content

Rename Lamdera Platform initializer - #1

Open
derrickbeining wants to merge 1 commit into
lamdera:lamderafrom
derrickbeining:fix/platform-initialize-lamdera-name
Open

Rename Lamdera Platform initializer#1
derrickbeining wants to merge 1 commit into
lamdera:lamderafrom
derrickbeining:fix/platform-initialize-lamdera-name

Conversation

@derrickbeining

Copy link
Copy Markdown

Summary

  • Rename Lamdera's replacement Platform initializer from _Platform_initialize to _Platform_initialize_Lamdera.
  • Update the replacement Platform worker call site to use the Lamdera-specific initializer name.

Part of lamdera/compiler#97.

Why

The Lamdera compiler output can include both the standard compiler runtime _Platform_initialize and this replacement elm/core initializer. Because both declarations use the same global name, the later replacement-package declaration can override downstream tooling patches to the standard runtime initializer, such as those made by elm-watch in Elm Land dev builds.

Giving the replacement initializer a Lamdera-specific name avoids that collision while preserving the replacement runtime behavior.

Test plan

  • Not run in this package repo directly.
  • Verified downstream that the collision comes from the duplicate _Platform_initialize emitted by the replacement Platform kernel.

Made with Cursor

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.

1 participant