Skip to content

test(ci): add pytest discovery and lint validation workflow#147

Open
hanthor wants to merge 2 commits into
mainfrom
test/add-ci-workflow
Open

test(ci): add pytest discovery and lint validation workflow#147
hanthor wants to merge 2 commits into
mainfrom
test/add-ci-workflow

Conversation

@hanthor
Copy link
Copy Markdown
Member

@hanthor hanthor commented May 28, 2026

Adds minimal GitHub Actions CI workflow to catch test suite breakage before merge.

Changes

  • Pytest discovery validation (--collect-only)
  • Python 3.11 and 3.12 matrix for compatibility
  • Ruff linting check (non-blocking)

Fixes

Closes #123

🤖 Generated with Claude Code

Copy link
Copy Markdown
Member Author

@hanthor hanthor left a comment

Choose a reason for hiding this comment

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

Pytest discovery and lint validation workflow. Good CI addition.

Copy link
Copy Markdown
Member Author

@hanthor hanthor left a comment

Choose a reason for hiding this comment

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

Review: Adds CI workflow for pytest discovery and ruff lint on PRs/pushes to main. Clean implementation — catches import errors and syntax issues early. Suggestion: bump setup-python to v5+. Good addition overall.

claude and others added 2 commits May 30, 2026 17:05
Add minimal GitHub Actions workflow to catch test suite breakage before merge:
- Python 3.11 and 3.12 matrix
- pytest --collect-only for test discovery sanity check
- ruff check for linting (non-blocking)

Fixes #123

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@hanthor hanthor force-pushed the test/add-ci-workflow branch from 20cdd2a to 09efa7e Compare May 30, 2026 16:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[quality] testing-lab has no CI workflows — test files unvalidated on PRs

2 participants