Skip to content

[DEV] Introduce ElmishProgram wrapper lifecycle#8

Open
lukaszkrzywizna wants to merge 6 commits into
mainfrom
codex/elmish-program-wrapper
Open

[DEV] Introduce ElmishProgram wrapper lifecycle#8
lukaszkrzywizna wants to merge 6 commits into
mainfrom
codex/elmish-program-wrapper

Conversation

@lukaszkrzywizna
Copy link
Copy Markdown
Collaborator

Summary

  • replace the post-map style debugger integration with an ElmishProgram setup/run wrapper
  • keep the prepared program immutable from the outside while storing runtime state in closures
  • add convenience createStore helpers so callers can choose split or simple store creation

Validation

  • dotnet build ElmishStore.sln
  • dotnet fable ./src/ElmishStore.Example --define DEBUG --outDir ./src/ElmishStore.Example/.fable-build
  • manual browser smoke on http://localhost:8080 with mocked Redux DevTools jump-to-state and jump-to-action

@tailwind components;
@tailwind utilities; No newline at end of file
@import "tailwindcss";
@source "../.fable-build";
Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

explain that change

Comment thread .config/dotnet-tools.json Outdated
"tools": {
"fable": {
"version": "4.9.0",
"version": "5.0.0-rc.7",
Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

There is an official fable 5 already published.

Comment thread postcss.config.js Outdated
export default {
plugins: {
tailwindcss: {},
"@tailwindcss/postcss": {},
Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

Do we still need postcss for tailwind 4?

Comment thread tailwind.config.js Outdated
],
plugins: []
}
export default {}
Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

if empty then just remove it

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