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

readme: update no_std notes in readme; ci: add no_std test #23

Merged
merged 3 commits into from
Jan 10, 2025

Conversation

hsqStephenZhang
Copy link
Contributor

background

before(include) 0.2.8, it won't compile on no_std target with the following config:

caches = { version = "0.2.8" ,default-features = false, features = ["hashbrown"] }

since 0.2.9 is not published, the current work around is:

caches = { git = "https://github.com/al8n/caches-rs.git", branch = "main", default-features = false, features = ["hashbrown", "libm"] }

this should also be pointed out in readme
This pull request includes updates to the CI workflow and documentation to improve testing and clarify feature usage for the caches crate.

CI Workflow Updates:

Documentation Updates:

  • README.md: Updated the no_std section to clarify the compilation requirements for versions before and after 0.2.8, and provided instructions for using the hashbrown and libm features with version 0.2.9 or the main branch from the repository.

todo

  • publish 0.2.9 so it can work with no_std

@al8n
Copy link
Owner

al8n commented Jan 10, 2025

I just bumped up some versions, and increased the version to 0.3.x, are you willing to update this PR? I will publish 0.3.x once this PR is merged.

@hsqStephenZhang
Copy link
Contributor Author

yeah, i'd be glad to

@al8n al8n merged commit 737f3c7 into al8n:main Jan 10, 2025
23 of 31 checks passed
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