fix: restore missing enum and fix import ordering after web_time migration#259
Draft
fix: restore missing enum and fix import ordering after web_time migration#259
Conversation
Adds a `time_stores` feature (included in default) that gates all time-dependent cache types (TimedCache, TimedSizedCache, ExpiringSizedCache). Users targeting wasm32-unknown-unknown without WASI/JS can now disable this feature to avoid the web-time dependency. Changes: - Add `time_stores` feature to Cargo.toml (in default features) - Make `web-time` an optional dependency gated by `time_stores` - Move Status enum from timed.rs to stores/mod.rs (shared) - Gate timed, timed_sized, expiring_sized modules behind time_stores - Fix once.rs proc macro to only generate web_time::Instant::now() when time= is set - Update doc tests and integration tests with cfg guards - Update documentation in lib.rs and README.md Co-authored-by: jaemk <13563490+jaemk@users.noreply.github.com>
Copilot
AI
changed the title
[WIP] Fix issue number 257
Add Mar 9, 2026
time_stores feature to gate time-dependent caches (fixes wasm32-unknown-unknown builds)
…and tests Co-authored-by: jaemk <13563490+jaemk@users.noreply.github.com>
Copilot
AI
changed the title
Add
Add Mar 9, 2026
time_stores feature to gate time-dependent caches (fixes wasm32-unknown-unknown builds)time_stores feature to gate time-dependent cache types for bare WASM targets
Co-authored-by: jaemk <13563490+jaemk@users.noreply.github.com>
Copilot
AI
changed the title
Add
Replace all std::time usage with web_time across the codebase
Mar 9, 2026
time_stores feature to gate time-dependent cache types for bare WASM targets…dependency note Co-authored-by: jaemk <13563490+jaemk@users.noreply.github.com>
Copilot
AI
changed the title
Replace all std::time usage with web_time across the codebase
Replace all Mar 9, 2026
std::time usage with web_time drop-in replacement; sync README
Copilot
AI
changed the title
Replace all
docs: sync README.md with src/lib.rs via cargo-readme
Mar 9, 2026
std::time usage with web_time drop-in replacement; sync README… ordering Co-authored-by: jaemk <13563490+jaemk@users.noreply.github.com>
Copilot
AI
changed the title
docs: sync README.md with src/lib.rs via cargo-readme
fix: restore missing enum and fix import ordering after web_time migration
Mar 9, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Two regressions introduced during the
std::time→web_timemigration broke CI.Changes
examples/redis-async.rs— Restored accidentally deleted#[derive(Error, Debug, PartialEq, Clone)] enum ExampleErrordeclaration, which left orphanedRedisError(String),and}causing a parse errorrustfmtimport ordering:web_time(w) imports were placed before alphabetically-later crates (thiserror,tokio) inexamples/,src/stores/, andtests/✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.