Skip to content

Fix/trunk wasm start#74

Open
syzer wants to merge 10 commits into
rs4rse:mainfrom
syzer:fix/trunk-wasm-start
Open

Fix/trunk wasm start#74
syzer wants to merge 10 commits into
rs4rse:mainfrom
syzer:fix/trunk-wasm-start

Conversation

@syzer

@syzer syzer commented Mar 5, 2026

Copy link
Copy Markdown
Collaborator

No description provided.

@syzer syzer force-pushed the fix/trunk-wasm-start branch from 8c5e823 to 7550753 Compare March 5, 2026 13:35
@unkcpz

unkcpz commented Mar 5, 2026

Copy link
Copy Markdown
Member

The typing and structure selection not works well on iPhone sadly. I'll try to fix it, this will be a good exercise for me ;)

Comment on lines +33 to +37
- name: Run wasm compile checks
run: |
cargo check -p vizmat-core --target wasm32-unknown-unknown --no-default-features
cargo check -p vizmat-app --target wasm32-unknown-unknown --no-default-features

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I assume wasm-pack build will build it with wasm32 target no?

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.

this is faster to fail invalid wasm . no need to run build if that fails

Comment thread .github/workflows/ci.yml
run: cargo fmt --all -- --check

# Run wasm compile checks to catch web-only type mismatches.
wasm_check:

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

This is covered by deploy.yml, so I think no need to duplicate test it here?

Emm... I think that deploy test is a bit more, so I get you now. Make sense to test build for wasm here in CI.

@syzer

syzer commented Mar 5, 2026

Copy link
Copy Markdown
Collaborator Author

The typing and structure selection not works well on iPhone sadly. I'll try to fix it, this will be a good exercise for me ;)

was thinking to switching for web selectbox even .. because we have to handle events manually.
and values are chosen arbitrary

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.

2 participants