Skip to content

Commit

Permalink
Update push-test.yml
Browse files Browse the repository at this point in the history
When the upload-artifact action was updated to v4, the behavior of it changes such that you cannot create a unified "artifact" by having it add files to an existing artifact name. Instead, they all require a unique artifact name to be uploaded.

This should fix the current workflow build errors.
  • Loading branch information
daxpryce authored Oct 15, 2024
1 parent 6f2691c commit f5138ba
Showing 1 changed file with 3 additions and 6 deletions.
9 changes: 3 additions & 6 deletions .github/workflows/push-test.yml
Original file line number Diff line number Diff line change
Expand Up @@ -35,19 +35,16 @@ jobs:
with:
fetch-depth: 1
submodules: true
- name: Build dispannpy dependency tree
- name: Build diskannpy dependency tree
run: |
pip install diskannpy pipdeptree
echo "dependencies" > dependencies_${{ matrix.os }}.txt
pipdeptree >> dependencies_${{ matrix.os }}.txt
- name: Archive dispannpy dependencies artifact
- name: Archive diskannpy dependencies artifact
uses: actions/upload-artifact@v4
with:
name: dependencies
name: dependencies_${{ matrix.os }}
path: |
dependencies_${{ matrix.os }}.txt
- name: DiskANN Build CLI Applications
uses: ./.github/actions/build
# python:
# name: DiskANN Build Python Wheel
# uses: ./.github/workflows/build-python.yml

0 comments on commit f5138ba

Please sign in to comment.