Skip to content

remove core and server from javadocs#1911

Merged
mcm001 merged 2 commits intoPhotonVision:mainfrom
samfreund:remove-javadocs
Apr 17, 2025
Merged

remove core and server from javadocs#1911
mcm001 merged 2 commits intoPhotonVision:mainfrom
samfreund:remove-javadocs

Conversation

@samfreund
Copy link
Member

@samfreund samfreund commented Apr 17, 2025

Description

The javadocs should reflect only what is shipped in the vendordep. Currently, photon-core and photon-server are not shipped with the vendordep, therefore they are being removed from the javadoc.

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 v2024.3.1
  • 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

@samfreund samfreund requested a review from a team as a code owner April 17, 2025 15:15
@mcm001
Copy link
Contributor

mcm001 commented Apr 17, 2025

Still depends on a photon core task? Line 124

@mcm001 mcm001 enabled auto-merge (squash) April 17, 2025 15:26
@mcm001 mcm001 merged commit b8d7452 into PhotonVision:main Apr 17, 2025
64 of 65 checks passed
@samfreund samfreund deleted the remove-javadocs branch May 2, 2025 20:12
samfreund added a commit to samfreund/photonvision that referenced this pull request Jun 28, 2025
@Gold856 Gold856 added the documentation Anything relating to https://docs.photonvision.org label Jul 12, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Anything relating to https://docs.photonvision.org

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants