Skip to content

Clean up client dependencies#1954

Merged
samfreund merged 11 commits intoPhotonVision:mainfrom
GrahamSH-LLK:upgrade-deps
May 7, 2025
Merged

Clean up client dependencies#1954
samfreund merged 11 commits intoPhotonVision:mainfrom
GrahamSH-LLK:upgrade-deps

Conversation

@GrahamSH-LLK
Copy link
Contributor

@GrahamSH-LLK GrahamSH-LLK commented May 6, 2025 •
edited by Gold856
Loading

Description

Updates all client dependencies as many were outdated and removes unused dependencies.

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

@GrahamSH-LLK GrahamSH-LLK requested a review from a team as a code owner May 6, 2025 23:51
@samfreund

This comment was marked as off-topic.

Sign in to view
mcm001

This comment was marked as resolved.

Sign in to view

@GrahamSH-LLK @samfreund
Update linting in workflows
2da46f9 
Update workflows

fix linting

oops, gotta change it here too

Discard changes to photon-server/src/main/resources/web/index.html

actually do the linting
@samfreund samfreund dismissed mcm001’s stale review May 7, 2025 00:34

resolved, docs r updated

mcm001
mcm001 reviewed May 7, 2025
View reviewed changes
@samfreund
Copy link
Member

samfreund commented May 7, 2025

Dropping unused dependencies still needs to be done.

@Gold856 Gold856 force-pushed the upgrade-deps branch 5 times, most recently from c48a079 to e852497 Compare May 7, 2025 04:53
@Gold856 Gold856 force-pushed the upgrade-deps branch 4 times, most recently from df97d30 to cffd4a5 Compare May 7, 2025 05:55
@Gold856 Gold856 changed the title Update all client dependencies Clean up client dependencies May 7, 2025
@yuhao1118 yuhao1118 mentioned this pull request May 7, 2025
Closed
7 tasks
mcm001
mcm001 approved these changes May 7, 2025
View reviewed changes
Copy link
Contributor

@mcm001 mcm001 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

we ball

@samfreund samfreund merged commit 74322af into PhotonVision:main May 7, 2025
39 checks passed
yuhao1118 pushed a commit to ATOM-STORM-6766/photonvision that referenced this pull request May 7, 2025
@GrahamSH-LLK @samfreund
@Gold856 @yuhao1118
Clean up client dependencies (PhotonVision#1954)
f37c58d 
Co-authored-by: samfreund <[email protected]>
Co-authored-by: Gold856 <[email protected]>
samfreund added a commit to samfreund/photonvision that referenced this pull request Jun 28, 2025
@GrahamSH-LLK @samfreund @Gold856
Clean up client dependencies (PhotonVision#1954)
40f7ff2 
Co-authored-by: samfreund <[email protected]>
Co-authored-by: Gold856 <[email protected]>
@Gold856 Gold856 added the frontend Having to do with PhotonClient and its related items label Jul 12, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@samfreund samfreund samfreund left review comments

@mcm001 mcm001 mcm001 approved these changes

Assignees

No one assigned

Labels

frontend Having to do with PhotonClient and its related items

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@GrahamSH-LLK @samfreund @mcm001 @Gold856