Skip to content

Require quality checks and update contributor tooling - #517

Draft
gijzelaerr wants to merge 1 commit into
spotify:masterfrom
gijzelaerr:quality/ci-contributor-workflow
Draft

gijzelaerr wants to merge 1 commit into
spotify:masterfrom
gijzelaerr:quality/ci-contributor-workflow

Conversation

@gijzelaerr

Copy link
Copy Markdown
Member

Summary

  • Require Python lint, type checking, and C++ formatting jobs to pass.
  • Make the editor subprocess test launch the active Python interpreter, so virtual environments work correctly.
  • Update the documented Python minimum and the contributor build commands to match scikit-build-core and CMake.
  • Replace obsolete tox commands with the Ruff checks used in CI.

The contributor-documentation changes overlap #497 and parts of #502; the tooling changes also touch work proposed in #503. These should be reconciled during review.

Validation

  • Native build and full test suite passed locally.
  • Pyright, Ruff lint and format, C++ formatting, and workflow YAML parsing passed locally.

This branch has not been deployed

No deployments
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