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.

Sign up for GitHub

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

Jump to bottom

fix: cache key stability #142

Merged
merged 2 commits into from May 18, 2023
Merged

fix: cache key stability #142

merged 2 commits into from May 18, 2023

Commits on May 17, 2023

  1. fix: cache key stability 

    Ensure consistency of main and post configuration by storing and
restoring it from state, which in turn ensures cache key stability.

Also:
* Fixed some typos.
* Use core.error for logging errors.
* Fix inverted condition on cache-all-crates.

Reverts: Swatinem#138
Fixes Swatinem#140
    @stevenh
    stevenh committed May 17, 2023
    Copy the full SHA
    68377a4 View commit details
    Browse the repository at this point in the history

  2. chore: make config load from state explicit 

    Make the loading of config from state explicit by splitting into a
dedicated static method.
    @stevenh
    stevenh committed May 17, 2023
    Copy the full SHA
    c7502cd View commit details
    Browse the repository at this point in the history