Skip to content

Use target family insted of target arch#2975

Merged
timon-schelling merged 2 commits intomasterfrom
use-target-family-insted-of-target-arch
Aug 3, 2025
Merged

Use target family insted of target arch#2975
timon-schelling merged 2 commits intomasterfrom
use-target-family-insted-of-target-arch

Conversation

@timon-schelling
Copy link
Copy Markdown
Member

primarily

- cfg(target_arch = "wasm32")
+ cfg(target_family = "wasm")

see https://doc.rust-lang.org/reference/conditional-compilation.html#target_family

plus fixes for warnings in test builds, that slipped trough in my last pr, sorry for that

@github-actions github-actions Bot temporarily deployed to graphite-dev (Preview) August 2, 2025 15:34 Inactive
@timon-schelling timon-schelling force-pushed the use-target-family-insted-of-target-arch branch from b6d1494 to 599f853 Compare August 2, 2025 16:18
@github-actions github-actions Bot temporarily deployed to graphite-dev (Preview) August 2, 2025 16:21 Inactive
@timon-schelling timon-schelling force-pushed the use-target-family-insted-of-target-arch branch from 599f853 to 847a904 Compare August 2, 2025 16:34
@github-actions github-actions Bot temporarily deployed to graphite-dev (Preview) August 2, 2025 16:37 Inactive
@timon-schelling timon-schelling force-pushed the use-target-family-insted-of-target-arch branch from 847a904 to 05f5383 Compare August 2, 2025 16:43
@github-actions github-actions Bot temporarily deployed to graphite-dev (Preview) August 2, 2025 16:47 Inactive
@timon-schelling timon-schelling force-pushed the use-target-family-insted-of-target-arch branch from 05f5383 to ac1af66 Compare August 3, 2025 10:03
@timon-schelling timon-schelling enabled auto-merge (squash) August 3, 2025 10:04
@github-actions github-actions Bot temporarily deployed to graphite-dev (Preview) August 3, 2025 10:16 Inactive
@timon-schelling timon-schelling merged commit 67123f5 into master Aug 3, 2025
4 checks passed
@timon-schelling timon-schelling deleted the use-target-family-insted-of-target-arch branch August 3, 2025 10:28
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