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

Allow euclid to be built in no_std environments #289

Merged
merged 1 commit into from Jun 4, 2018
Merged

Allow euclid to be built in no_std environments #289

merged 1 commit into from Jun 4, 2018

Conversation

GabrielMajeri
Copy link
Contributor

@GabrielMajeri GabrielMajeri commented Jun 4, 2018

(blocked on #279, since only num-traits > 0.2 is no_std compatible)

Since this crate does not use any std-specific features, this pull request makes it no_std.

The tests are still built and run using std.


This change is Reviewable

Copy link
Member

@kvark kvark left a comment

Choose a reason for hiding this comment

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

looks reasonable, thank you!

@jdm
Copy link
Member

jdm commented Jun 4, 2018

@bors-servo r=kvark

@bors-servo
Copy link
Contributor

📌 Commit a462d6a has been approved by kvark

@bors-servo
Copy link
Contributor

⌛ Testing commit a462d6a with merge d155e0c...

bors-servo pushed a commit that referenced this pull request Jun 4, 2018
Allow euclid to be built in no_std environments

(blocked on #279, since only `num-traits` > 0.2 is `no_std` compatible)

Since this crate does not use any `std`-specific features, this pull request makes it `no_std`.

The tests are still built and run using `std`.

<!-- Reviewable:start -->
---
This change is [<img src="https://reviewable.io/review_button.svg" height="34" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/euclid/289)
<!-- Reviewable:end -->
@jdm
Copy link
Member

jdm commented Jun 4, 2018

Sorry, overlooked that it depends on another PR.

@bors-servo
Copy link
Contributor

☀️ Test successful - status-travis
Approved by: kvark
Pushing d155e0c to master...

@bors-servo bors-servo merged commit a462d6a into servo:master Jun 4, 2018
@GabrielMajeri
Copy link
Contributor Author

It should be alright, it works and the tests pass but it won't be usable in no_std environments until the other one gets merged.

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.

None yet

4 participants