Skip to content

Commit

Permalink
chore: run pre-commit install in CI step
Browse files Browse the repository at this point in the history
Signed-off-by: Jennifer Power <[email protected]>
  • Loading branch information
jpower432 committed Oct 4, 2023
1 parent f6e2c7f commit e3a98b6
Show file tree
Hide file tree
Showing 2 changed files with 12 additions and 5 deletions.
10 changes: 7 additions & 3 deletions .github/workflows/ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,8 @@ name: CI
on:
push:
pull_request:
branches: main
branches:
- main

concurrency:
group: ${{ github.ref }}-${{ github.workflow }}
Expand Down Expand Up @@ -36,11 +37,14 @@ jobs:

- name: Set up poetry and install
uses: ./.github/actions/setup-poetry

- name: Pre-commit install
run: make pre-commit

- name: Run checks
- name: Run linting checks
run: make lint

- name: Run bandit
- name: Run security checks
run: make security-check

- name: Check dependencies
Expand Down
7 changes: 5 additions & 2 deletions Makefile
Original file line number Diff line number Diff line change
Expand Up @@ -4,11 +4,14 @@ TESTS := tests
all: develop lint test
.PHONY: all

develop:
develop: pre-commit
@poetry install
@poetry shell
.PHONY: develop

pre-commit:
@poetry run pre-commit install

lint:
@poetry lock --check
@poetry run isort --profile=black --lines-after-imports=2 \
Expand Down Expand Up @@ -38,7 +41,7 @@ dep-cve-check:
.PHONY: dep-cve-check

security-check:
@pre-commit run semgrep --all-files
@poetry run pre-commit run semgrep --all-files
.PHONY: security-check

build: clean-build
Expand Down

0 comments on commit e3a98b6

Please sign in to comment.