Skip to content

Commit

Permalink
Actually, don't run pre-commit in nox
Browse files Browse the repository at this point in the history
  • Loading branch information
object-Object committed Nov 5, 2023
1 parent bbf13d8 commit c55d8ca
Show file tree
Hide file tree
Showing 3 changed files with 5 additions and 7 deletions.
5 changes: 4 additions & 1 deletion .github/workflows/ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -22,8 +22,11 @@ jobs:
with:
python-version: '3.11'

- name: Run pre-commit
uses: pre-commit/[email protected]

- name: Install Nox
run: pip install nox

- name: Run tests
- name: Run Nox
run: nox
4 changes: 0 additions & 4 deletions .vscode/settings.json
Original file line number Diff line number Diff line change
Expand Up @@ -10,10 +10,6 @@
"[yaml][html][css][javascript][jinja-html][jinja-css][jinja-js]": {
"editor.tabSize": 2,
},
"[yaml]": {
"editor.defaultFormatter": "redhat.vscode-yaml",
"editor.formatOnSave": true,
},
"[html][jinja-html]": {
"editor.rulers": [120],
},
Expand Down
3 changes: 1 addition & 2 deletions noxfile.py
Original file line number Diff line number Diff line change
Expand Up @@ -5,8 +5,7 @@
def tests(session: nox.Session):
session.install("-e", ".[test]", "-e", "./test/_submodules/HexMod")

session.run("pre-commit")
session.run("pytest", *session.posargs)

# 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 c55d8ca

Please sign in to comment.