Skip to content

chore(py_pytest_main): Refactor to allow for linting #570

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.

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

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

arrdem
Copy link
Collaborator

@arrdem arrdem commented May 30, 2025

The pytest driver template isn't valid Python, which makes it unlintable, which is part of why it has unmitigated typing issues which become visible when users adopt rules_mypy and equivalents.

Rework the driver template so that in its pre-expansion form it's valid Python and impose linting on it.


Changes are visible to end-users: no

Test plan

TBD

Copy link

aspect-workflows bot commented May 30, 2025

Test

All tests were cache hits

35 tests (100.0%) were fully cached saving 1m 3s.

Copy link

e2e/use_release folder: LCOV of commit c261808 during CI #1796

Summary coverage rate:
  lines......: 100.0% (2 of 2 lines)
  functions..: 100.0% (1 of 1 function)
  branches...: no data found

Files changed coverage rate: n/a

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.

1 participant