Skip to content

Commit

Permalink
chore: add missing deps to Pipfile and fix a typo
Browse files Browse the repository at this point in the history
Signed-off-by: Tomas Coufal <[email protected]>
  • Loading branch information
tumido committed Oct 14, 2024
1 parent 71c601c commit 3a0e4a5
Show file tree
Hide file tree
Showing 5 changed files with 229 additions and 115 deletions.
10 changes: 7 additions & 3 deletions .github/workflows/pre_commit.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -20,23 +20,27 @@ jobs:
uses: actions/setup-python@v5
with:
python-version: "3.12"
cache: pipenv

- name: Setup Pipenv
uses: tiagovrtr/actions-pipenv@v1

- name: Configure caching
uses: actions/cache@v4
with:
path: ~/.cache/pre-commit
key: precommit-${{ runner.os }}-${{ hashFiles('.pre-commit-config.yaml') }}

- name: Install pre-commit
- name: Install dependencies
run: |
pip install pre-commit
python -m ensurepip --upgrade
pipenv sync
- name: Run pre-commit
run: |
pre-commit run --all-files
- name: Test if pipeline is up-to-date
run: |
pip install click kfp==2.9.0 kfp.kubernetes
make pipeline
git diff --exit-code || (echo "Pipeline is not up-to-date. Please run 'make pipeline' and commit the changes." && exit 1)
4 changes: 4 additions & 0 deletions Pipfile
Original file line number Diff line number Diff line change
Expand Up @@ -6,6 +6,10 @@ name = "pypi"
[packages]
kfp = {extras = ["kubernetes"], version = "*"}
click = "*"
ruff = "*"
pyyaml = "*"
kubernetes = "*"
pre-commit = "*"

[dev-packages]

Expand Down
Loading

0 comments on commit 3a0e4a5

Please sign in to comment.