Skip to content

Conglomerate release artifacts#2115

Merged
samfreund merged 2 commits intoPhotonVision:mainfrom
samfreund:bump-gh-release-action
Oct 11, 2025
Merged

Conglomerate release artifacts#2115
samfreund merged 2 commits intoPhotonVision:mainfrom
samfreund:bump-gh-release-action

Conversation

@samfreund
Copy link
Member

Description

The bug that required us to split artifacts as a workaround has been resolved, so we're lumping everything back together.

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

@samfreund samfreund requested a review from a team as a code owner October 10, 2025 03:41
@samfreund samfreund requested a review from crschardt October 10, 2025 03:41
Copy link
Contributor

@crschardt crschardt left a comment

Choose a reason for hiding this comment

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

If this works on photon-image-modifier, I'll approve (or anyone else can).

Copy link
Member

@Gold856 Gold856 left a comment

Choose a reason for hiding this comment

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

Where's the action update?

@samfreund samfreund force-pushed the bump-gh-release-action branch from d045bcd to 33ebfd0 Compare October 10, 2025 05:13
@samfreund samfreund requested a review from Gold856 October 10, 2025 05:13
@samfreund
Copy link
Member Author

If this works on photon-image-modifier, I'll approve (or anyone else can).

@crschardt works.

@mcm001
Copy link
Contributor

mcm001 commented Oct 11, 2025

(if it does, please attach to a link to the passing run)

@samfreund
Copy link
Member Author

@samfreund samfreund merged commit 8ec0414 into PhotonVision:main Oct 11, 2025
40 checks passed
@crschardt
Copy link
Contributor

We have to push a tag on the repo to actually test this. Dev images don't exercise the releaser. See here in the latest CI run on main in photon-image-modifier. https://github.com/PhotonVision/photon-image-modifier/actions/runs/18420645394/job/52496956941#step:6:1

@samfreund samfreund deleted the bump-gh-release-action branch October 20, 2025 21:48
spacey-sooty pushed a commit to spacey-sooty/photonvision that referenced this pull request Dec 22, 2025
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.

4 participants