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

Abstract out provisioning across all component types #340

Merged
merged 4 commits into from Mar 24, 2024
Merged

Conversation

whitfin
Copy link
Owner

@whitfin whitfin commented Mar 23, 2024

This fixes #332.

This PR will introduce a new behaviour and service based on provisioning. Instead of it being part of hooks, as before, it's now a separate Cachex.Provision module which can be used in any component to accept provisions. Most notable this is now also supported in warmers, so they can have an up to date version of the cache structure.

This PR will also name the cache warmers after their module. This is temporary, because eventually they'll be named (via #334) - but naming was required to implement this properly. It was only not done in the past due to not being tied to a major release.

@whitfin whitfin added this to the v4.0.0 milestone Mar 23, 2024
@whitfin whitfin self-assigned this Mar 23, 2024
@whitfin whitfin merged commit b22cb1f into main Mar 24, 2024
15 checks passed
@whitfin whitfin deleted the issue-332 branch March 24, 2024 02:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Consider upgrading provisions to support Warmers as well as Hooks
1 participant