Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

implement floor and ceil for f64 and f32 types #183

Merged
merged 7 commits into from
Dec 3, 2024

Conversation

kralverde
Copy link
Contributor

This PR implements the floor and ceil functions for the f64 and f32 types.

@kralverde kralverde marked this pull request as ready for review November 29, 2024 05:52
@kralverde kralverde marked this pull request as draft November 29, 2024 16:59
@kralverde kralverde marked this pull request as ready for review November 29, 2024 17:32
@kralverde
Copy link
Contributor Author

It looks like the workflow is broken for WASM other than stable, but running all other workflows succeeded for me locally.

@CryZe
Copy link
Contributor

CryZe commented Dec 2, 2024

The WASI target is now called wasm32-wasip1

@kralverde
Copy link
Contributor Author

The WASI target is now called wasm32-wasip1

Thanks this worked, but now it looks like there's some kind of permission issue with the workflows...

@kralverde
Copy link
Contributor Author

Now an architecture error... I'm not super familiar with wasm if anyone wants to pitch in

@Lokathor
Copy link
Owner

Lokathor commented Dec 3, 2024

I'm gonna merge this, the wasm stuff shouldn't be on your shoulders.

@CryZe can you fix the wasm ci setup at some point later?

@Lokathor Lokathor merged commit a9b883f into Lokathor:main Dec 3, 2024
7 of 26 checks passed
@CryZe
Copy link
Contributor

CryZe commented Dec 3, 2024

The problem is that it's still the old target name here (and possibly other places): https://github.com/Lokathor/wide/blob/main/.cargo-ci/config#L11

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

Successfully merging this pull request may close these issues.

3 participants