Skip to content

GitHub Action to spellcheck and lint Python code #333

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 6 commits into from
Apr 28, 2024
Merged

GitHub Action to spellcheck and lint Python code #333

Changes from 1 commit
Commits
Show all changes
6 commits
Select commit Hold shift + click to select a range
2c209b7
GitHub Action to spellcheck and lint Python code
cclauss Apr 27, 2024
ceeabbe
Update pyproject.toml
cclauss Apr 28, 2024
4a6c409
$(VENV)/bin/ruff check --fix
cclauss Apr 28, 2024
d30e65c
exclude: python-version: "3.7" on os: "macos-latest"
cclauss Apr 28, 2024
22be3ce
Update tests.yml
cclauss Apr 28, 2024
37289bb
Allow Python 3.7 on ARM to crash
cclauss Apr 28, 2024
File filter

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
exclude: python-version: "3.7" on os: "macos-latest"
  • Loading branch information
@cclauss
cclauss authored Apr 28, 2024
commit d30e65c59cf0922994b3f41ff5e2c840eb28f8a2
4 changes: 4 additions & 0 deletions .github/workflows/tests.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -18,6 +18,10 @@ jobs:
matrix:
python-version: ["3.7", "3.8", "3.9", "3.10", "3.11", "3.12"]
os: ["macos-latest", "windows-latest", "ubuntu-latest"]
exclude:
- python-version: "3.7"
os: "macos-latest"
Copy link
Member

@woodruffw woodruffw Apr 28, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Let's just leave this out for now -- I'd rather merge this with a broken CI (or wait a little longer for GHA to fix it) than have a matrix exclude rule that I'll likely forget about 🙂


steps:
- uses: "actions/checkout@v4"
Expand Down