Skip to content

Commit

Permalink
okay just don't run it then. fine
Browse files Browse the repository at this point in the history
  • Loading branch information
object-Object committed Nov 5, 2023
1 parent 59c6c0c commit 2ade7aa
Show file tree
Hide file tree
Showing 3 changed files with 9 additions and 7 deletions.
10 changes: 6 additions & 4 deletions .github/workflows/ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -23,11 +23,13 @@ jobs:
with:
python-version: '3.11'

- name: Install dependencies
run: pip install . nox

- name: Run pre-commit
- name: Run pre-commit hooks
env:
SKIP: pyright
uses: pre-commit/[email protected]

- name: Install Nox
run: pip install nox

- name: Run Nox
run: nox
2 changes: 0 additions & 2 deletions .pre-commit-config.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -18,5 +18,3 @@ repos:
- id: pyright
args: ["--warnings"]
files: '^src/'
additional_dependencies:
- "."
4 changes: 3 additions & 1 deletion noxfile.py
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,9 @@
def tests(session: nox.Session):
session.install("-e", ".[test]", "-e", "./test/_submodules/HexMod")

session.run("pytest", *session.posargs)
# this apparently CANNOT run from pre-commit in GitHub Actions (venv issues)
session.run("pyright", "src", "--warnings")

# test cookiecutter last so the extra package install doesn't interfere
session.run("pytest", *session.posargs)
session.run("pytest", "-k", "test_cookiecutter", "--nox", *session.posargs)

0 comments on commit 2ade7aa

Please sign in to comment.