forked from phoenix-oss/llama-stack-mirror
ci: Run pre-commit checks in CI (#176)
Run the pre-commit checks in a github workflow to validate that a PR or a direct push to the repo does not introduce new errors.
This commit is contained in:
parent
89d24a07f0
commit
eba9d1ea14
2 changed files with 50 additions and 5 deletions
10
.flake8
10
.flake8
|
@ -21,11 +21,11 @@ ignore =
|
||||||
optional-ascii-coding = True
|
optional-ascii-coding = True
|
||||||
exclude =
|
exclude =
|
||||||
./.git,
|
./.git,
|
||||||
./docs
|
./docs/*,
|
||||||
./build
|
./build,
|
||||||
./scripts,
|
./scripts,
|
||||||
./venv,
|
./venv,
|
||||||
*.pyi
|
*.pyi,
|
||||||
.pre-commit-config.yaml
|
.pre-commit-config.yaml,
|
||||||
*.md
|
*.md,
|
||||||
.flake8
|
.flake8
|
||||||
|
|
45
.github/workflows/pre-commit.yml
vendored
Normal file
45
.github/workflows/pre-commit.yml
vendored
Normal file
|
@ -0,0 +1,45 @@
|
||||||
|
name: Pre-commit
|
||||||
|
|
||||||
|
on: [pull_request]
|
||||||
|
|
||||||
|
jobs:
|
||||||
|
pre-commit:
|
||||||
|
runs-on: ubuntu-latest
|
||||||
|
|
||||||
|
steps:
|
||||||
|
- name: Checkout code
|
||||||
|
uses: actions/checkout@d632683dd7b4114ad314bca15554477dd762a938 # v4.2.0
|
||||||
|
|
||||||
|
- name: Set up Python
|
||||||
|
uses: actions/setup-python@f677139bbe7f9c59b41e40162b753c062f5d49a3 # v5.2.0
|
||||||
|
with:
|
||||||
|
python-version: '3.11'
|
||||||
|
cache: pip
|
||||||
|
cache-dependency-path: |
|
||||||
|
**/requirements*.txt
|
||||||
|
.pre-commit-config.yaml
|
||||||
|
|
||||||
|
- name: Install pre-commit
|
||||||
|
run: |
|
||||||
|
python -m pip install --upgrade pip
|
||||||
|
pip install pre-commit
|
||||||
|
|
||||||
|
- name: Fetch base branch
|
||||||
|
run: git fetch origin ${{ github.event.pull_request.base.ref }}:refs/remotes/origin/${{ github.event.pull_request.base.ref }}
|
||||||
|
|
||||||
|
- name: Fetch head commit from PR
|
||||||
|
run: git fetch origin pull/${{ github.event.pull_request.number }}/head:pr-${{ github.event.pull_request.number }}
|
||||||
|
|
||||||
|
- name: Get changed files
|
||||||
|
id: changed-files
|
||||||
|
run: |
|
||||||
|
git diff --name-only origin/${{ github.event.pull_request.base.ref }} pr-${{ github.event.pull_request.number }} > changed_files.txt
|
||||||
|
cat changed_files.txt
|
||||||
|
|
||||||
|
- name: Run pre-commit
|
||||||
|
run: |
|
||||||
|
if [ -s changed_files.txt ]; then
|
||||||
|
pre-commit run --files $(cat changed_files.txt | tr '\n' ' ')
|
||||||
|
else
|
||||||
|
echo "No changed files to run pre-commit on."
|
||||||
|
fi
|
Loading…
Add table
Add a link
Reference in a new issue