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
feat(http): Switch to pluggable HTTP cache implementation #27966
feat(http): Switch to pluggable HTTP cache implementation #27966
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can this be broken into smaller steps, or too hard?
Yes, let's split |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
superceeded 😉
Not really, now we have to switch to the new implementation. I would like to use this PR for this. |
# Conflicts: # lib/util/http/cache/abstract-http-cache-provider.ts # lib/util/http/cache/schema.ts # lib/util/http/index.ts # lib/util/http/types.ts
🎉 This PR is included in version 37.258.0 🎉 The release is available on: Your semantic-release bot 📦🚀 |
Changes
cacheProvider
option forHttp
classCacheProvider
interface for repository cacheHttp
request options typeThis PR makes it possible to connect package cache with the HTTP caching capabilities.
For example, npm caching logic could be simplified.
Context
Documentation (please check one with an [x])
How I've tested my work (please select one)
I have verified these changes via: