Re-enabling caches for build (5h + seems too long and issue seems

solved) actions/runner-images#572
huggingface · Dec 15, 2021 · e0a5931 · e0a5931
1 parent 0589dc6
commit e0a5931
Showing 1 changed file with 12 additions and 14 deletions.
diff --git a/.github/workflows/python.yml b/.github/workflows/python.yml
@@ -61,20 +61,17 @@ jobs:
           toolchain: stable
           components: rustfmt, clippy
 
-          # # Necessary for now for the cargo cache: https://github.com/actions/cache/issues/133#issuecomment-599102035
-          # - run: sudo chown -R $(whoami):$(id -ng) ~/.cargo/
-
-          # - name: Cache Cargo Registry
-          #   uses: actions/cache@v1
-          #   with:
-          #     path: ~/.cargo/registry
-          #     key: ${{ runner.os }}-cargo-registry-${{ hashFiles('**/Cargo.lock') }}
-
-          # - name: Cache Cargo Build Target
-          #   uses: actions/cache@v1
-          #   with:
-          #     path: ./bindings/python/target
-          #     key: ${{ runner.os }}-cargo-python-build-${{ hashFiles('**/Cargo.lock') }}
+          - name: Cache Cargo Registry
+            uses: actions/cache@v1
+            with:
+              path: ~/.cargo/registry
+              key: ${{ runner.os }}-cargo-registry-${{ hashFiles('**/Cargo.lock') }}
+
+          - name: Cache Cargo Build Target
+            uses: actions/cache@v1
+            with:
+              path: ./bindings/python/target
+              key: ${{ runner.os }}-cargo-python-build-${{ hashFiles('**/Cargo.lock') }}
 
       - name: Lint with RustFmt
         uses: actions-rs/cargo@v1
@@ -100,6 +97,7 @@ jobs:
         run: |
           python -m venv .env
           source .env/bin/activate
+          pip install -U pip
           pip install pytest requests setuptools_rust numpy "pyarrow<3.0.0" datasets
           python setup.py develop