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

fix: adds environment variable to disable structure caching #645

Merged
merged 1 commit into from
Feb 22, 2024

Conversation

jaydublu2002
Copy link

@jaydublu2002 jaydublu2002 commented Jan 22, 2024

Suggestion to resolve #641 - adds config and environment variable to disable structure caching that was introduced in v3.11.0

Disabling caching could be useful in development environments, and also in tests where you might want to mock the Cache facade without also having to mock the various calls to Cache made by this package.

@rubenvanassche
Copy link
Member

Added in v4

@jaydublu2002 jaydublu2002 changed the base branch from main to v3 February 22, 2024 08:10
@jaydublu2002
Copy link
Author

@rubenvanassche I don't know if switching this PR to merge to v3 branch is all that is needed to make this a fix for v3?

@rubenvanassche
Copy link
Member

Jup looks good

@rubenvanassche
Copy link
Member

Thanks

@rubenvanassche rubenvanassche merged commit 21b4d11 into spatie:v3 Feb 22, 2024
2 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.

Application tests that use Cache mocking fail since PR #622 (v3.11.0)
2 participants