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

Support no_std+alloc #8

Merged
merged 1 commit into from
Mar 3, 2024
Merged

Support no_std+alloc #8

merged 1 commit into from
Mar 3, 2024

Conversation

lu-zero
Copy link
Contributor

@lu-zero lu-zero commented Mar 3, 2024

Minimal changes needed to support using this crate as no_std+alloc.

Copy link
Contributor

@Tpt Tpt left a comment

Choose a reason for hiding this comment

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

Thank you very much!

default = ["std"]
std = ["serde?/std"]
alloc = ["serde?/alloc"]
serialize = ["dep:serde"]
Copy link
Contributor

Choose a reason for hiding this comment

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

This crate still have a MSRV of 1.58 so dep: is not supported

Copy link
Contributor

Choose a reason for hiding this comment

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

Fixed in #9 by upgrading the MSRV to 1.60

@Tpt Tpt merged commit ac33682 into oxigraph:main Mar 3, 2024
6 of 9 checks passed
@lu-zero lu-zero deleted the no_std branch March 3, 2024 15:53
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