google / ml-compiler-opt

Infrastructure for Machine Learning Guided Optimization (MLGO) in LLVM.
Apache License 2.0
630 stars 93 forks source link

Quick CI Fix #261

Closed boomanaiden154 closed 1 year ago

boomanaiden154 commented 1 year ago

The last patch removed the remaining requirements.txt as it was leftover from the pipenv transition. However, this broke the CI as setup-python's cache is set to pip which depends upon the hash of a requirements.txt. This implements a quick fix by setting the cache dependency path (used for the hash) to the Pipfile.lock. This is not optimal since the pipenv deps don't get cached (based on my testing), but they weren't before either (as far as I can tell). More testing is needed, but this should get the CI green.