Skip to content

Stop shipping unit tests in photonlibpy wheel#2309

Merged
mcm001 merged 1 commit intoPhotonVision:mainfrom
auscompgeek:patch-1
Jan 17, 2026
Merged

Stop shipping unit tests in photonlibpy wheel#2309
mcm001 merged 1 commit intoPhotonVision:mainfrom
auscompgeek:patch-1

Conversation

@auscompgeek
Copy link
Contributor

Description

The photonlibpy wheels currently contain a test package with everything in //photon-lib/py/test. This is most certainly not intended to be shipped to users on sys.path.

This package was being picked up due to this:

packages=find_packages(),

There's no reason for this test subdirectory to be a Python package, so remove the __init__.py.

Meta

Merge checklist:

  • Pull Request title is short, imperative summary of proposed changes
  • The description documents the what and why
  • If this PR changes behavior or adds a feature, user documentation is updated
  • If this PR touches photon-serde, all messages have been regenerated and hashes have not changed unexpectedly
  • If this PR touches configuration, this is backwards compatible with settings back to v2025.3.2
  • If this PR touches pipeline settings or anything related to data exchange, the frontend typing is updated
  • If this PR addresses a bug, a regression test for it is added

@auscompgeek auscompgeek requested a review from a team as a code owner January 17, 2026 13:36
@github-actions github-actions bot added the photonlib Things related to the PhotonVision library label Jan 17, 2026
@mcm001 mcm001 enabled auto-merge (squash) January 17, 2026 14:16
@mcm001 mcm001 merged commit 9011e28 into PhotonVision:main Jan 17, 2026
77 of 78 checks passed
@auscompgeek auscompgeek deleted the patch-1 branch January 17, 2026 23:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

photonlib Things related to the PhotonVision library

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants