KTI-2626: Enable cache redirector for Yarn & NPM#3188
Conversation
|
Looks like we need to re-generate the lock file because hashes are different now (why though?) |
|
@sandwwraith, that's because urls are now pointing to cache redirector. |
|
Discussed it elsewhere and agreed on setting up the redirector in the main branch, not only for |
|
@sandwwraith, before rebasing it all onto
WDYT? We can still start with unconditional cache redirection and see if there will be any complaints, and only then consider this approach. |
|
I don't remember many complaints about cache-redirector in 'big' Kotlin (although it does have a switch). IMO we can start without one |
7167672 to
e74251e
Compare
|
@sandwwraith, okay, I dropped the flag switching the redirector and rebased the branch onto |
kotlinx-serialization builds executed as part of the Kotlin aggregate build are failing intermittently due networking errors on the Yarn/Npm side. This PR sets up cache redirector.
Ideally, there would be a way to configure it for all projects maintained by the libraries team (such as coroutines, datetime, benchmarks, atomicfu, collections immutable) in a single place, but it seems to be tricky to implement. The main reason for that is reliance on APIs provided by KGP, those APIs depend on a particular KGP version and our libraries use different versions. Which may require slightly different approaches to configure redirection. So for now, it seems like the easiest fix would be making the proposed change.
^KTI-2626