Skip to content

[Task]: Use exclusion list in python precommit instead of inclusion list #24813

@damccorm

Description

@damccorm

What needs to happen?

With #24204 we are adopting a list of subfolders to run the python precommit on; it would be better to have a catch-all version that just excludes some directories we're sure we're testing. This is less likely to be brittle and miss tests we add in future new subdirectories.

Additional context - #24204 (comment)

Issue Priority

Priority: 2 (default / most normal work should be filed as P2)

Issue Components

  • Component: Python SDK
  • Component: Java SDK
  • Component: Go SDK
  • Component: Typescript SDK
  • Component: IO connector
  • Component: Beam examples
  • Component: Beam playground
  • Component: Beam katas
  • Component: Website
  • Component: Spark Runner
  • Component: Flink Runner
  • Component: Samza Runner
  • Component: Twister2 Runner
  • Component: Hazelcast Jet Runner
  • Component: Google Cloud Dataflow Runner

Metadata

Metadata

Assignees

Labels

P2done & doneIssue has been reviewed after it was closed for verification, followups, etc.pythontask

Type

No type

Projects

No projects

Relationships

None yet

Development

No branches or pull requests

Issue actions