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

Add tests #118

Merged
merged 9 commits into from
Nov 14, 2020
Merged

Add tests #118

merged 9 commits into from
Nov 14, 2020

Conversation

vinc
Copy link
Owner

@vinc vinc commented Nov 14, 2020

It's about time we added some tests right?

See: phil-opp/blog_os#591

@vinc
Copy link
Owner Author

vinc commented Nov 14, 2020

Compiling rand_chacha in test fail with the following error:

LLVM ERROR: Do not know how to split the result of this operator!

This seems related to rust-lang/rust#61721 that prevented us to build the crate in release until a few days ago, see #116

@vinc vinc merged commit 77b1865 into master Nov 14, 2020
@vinc vinc deleted the feature/tests branch November 14, 2020 22:06
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