CI: separate pre-commit github action job (#1457)
This commit is contained in:
parent
6555234dee
commit
b67bba590d
4
.github/workflows/build.yml
vendored
4
.github/workflows/build.yml
vendored
@ -26,8 +26,4 @@ jobs:
|
|||||||
shell: bash
|
shell: bash
|
||||||
create-venv-at-path: .venv
|
create-venv-at-path: .venv
|
||||||
- run: poetry run pytest tests --cov --no-cov-on-fail --cov-report=
|
- run: poetry run pytest tests --cov --no-cov-on-fail --cov-report=
|
||||||
- run: poetry run pyright reflex tests
|
|
||||||
- run: poetry run ruff check . --format github
|
|
||||||
- run: find reflex tests -name "*.py" -not -path reflex/reflex.py | xargs poetry run darglint
|
|
||||||
- run: poetry run black --check reflex tests
|
|
||||||
- run: poetry run coverage html
|
- run: poetry run coverage html
|
||||||
|
28
.github/workflows/pre-commit.yml
vendored
Normal file
28
.github/workflows/pre-commit.yml
vendored
Normal file
@ -0,0 +1,28 @@
|
|||||||
|
name: pre-commit
|
||||||
|
|
||||||
|
on:
|
||||||
|
pull_request:
|
||||||
|
branches: [main]
|
||||||
|
push:
|
||||||
|
# Note even though this job is called "pre-commit" and runs "pre-commit", this job will run
|
||||||
|
# also POST-commit on main also! In case there are mishandled merge conflicts / bad auto-resolves
|
||||||
|
# when merging into main branch.
|
||||||
|
branches: [main]
|
||||||
|
|
||||||
|
jobs:
|
||||||
|
pre-commit:
|
||||||
|
runs-on: ubuntu-latest
|
||||||
|
steps:
|
||||||
|
- uses: actions/checkout@v3
|
||||||
|
- uses: ./.github/actions/setup_build_env
|
||||||
|
with:
|
||||||
|
# running vs. one version of Python is OK
|
||||||
|
# i.e. ruff, black, etc.
|
||||||
|
python-version: 3.11
|
||||||
|
run-poetry-install: true
|
||||||
|
shell: bash
|
||||||
|
create-venv-at-path: .venv
|
||||||
|
# TODO pre-commit related stuff can be cached too (not a bottleneck yet)
|
||||||
|
- run: |
|
||||||
|
poetry run pip install pre-commit
|
||||||
|
poetry run pre-commit run --all-files
|
Loading…
Reference in New Issue
Block a user