Skip to content

Fix no_std#28

Merged
IvanUkhov merged 6 commits intostainless-steel:mainfrom
p-sira:main
Mar 2, 2026
Merged

Fix no_std#28
IvanUkhov merged 6 commits intostainless-steel:mainfrom
p-sira:main

Conversation

@p-sira
Copy link
Contributor

@p-sira p-sira commented Feb 28, 2026

This PR fixes no_std builds due to ellip's incorrect config.

Changes:

  • Update ellip to 1.1.0
  • For std feature flag, use ellip/std
  • For no_std feature flag, use ellip/libm
  • Add CI to check no_std builds

The previous build passed the test, because std leaked into the test. So, I recommend adding CI to check on targets that really have no std, which I included with this PR.

@IvanUkhov
Copy link
Member

Thank you! I would prefer to not have any new workflows. Let us remove that part for now.

@p-sira
Copy link
Contributor Author

p-sira commented Mar 2, 2026

Bump. I've removed the workflow.

@IvanUkhov IvanUkhov merged commit 1c08ecc into stainless-steel:main Mar 2, 2026
3 checks passed
@IvanUkhov
Copy link
Member

Thank you!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants