Skip to content

Refactor winget workflow to use wingetcreate CLI#1826

Merged
gtardif merged 4 commits intomainfrom
fix_winget_release
Feb 23, 2026
Merged

Refactor winget workflow to use wingetcreate CLI#1826
gtardif merged 4 commits intomainfrom
fix_winget_release

Conversation

@gtardif
Copy link
Contributor

@gtardif gtardif commented Feb 23, 2026

From #1825, allowing to test the winget release before merging the PR here

  • Allow to test workflow manually from a branch (on upstream)
  • Added a dry-run flag to allow testing workflow without submitting a PR

Signed-off-by: Lorena Rangel <lorena.rangel@docker.com>
@gtardif gtardif requested a review from a team as a code owner February 23, 2026 11:01
@gtardif gtardif marked this pull request as draft February 23, 2026 11:01
docker-agent[bot]

This comment was marked as resolved.

Signed-off-by: Guillaume Tardif <guillaume.tardif@gmail.com>
Signed-off-by: Guillaume Tardif <guillaume.tardif@gmail.com>
@gtardif
Copy link
Contributor Author

gtardif commented Feb 23, 2026

Workflow tested here => PR created in winget to update to v1.23.5.

@gtardif gtardif marked this pull request as ready for review February 23, 2026 12:44
trungutt
trungutt previously approved these changes Feb 23, 2026
token: ${{ steps.app-token.outputs.token }}
$Version = $ENV:VERSION
$DryRun = $ENV:DRY_RUN -eq 'true'
$PackageId = "Docker.Cagent"
Copy link
Contributor

Choose a reason for hiding this comment

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

should we already start migrating to Docker.Agent, or is that better leftfor another moment?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I'm planning this for a follow up, indeed, this is just fixing this workflow that had never been green before
(we'll need to create the Docker.Agent entry in winget first)

Signed-off-by: Guillaume Tardif <guillaume.tardif@gmail.com>
@gtardif gtardif requested a review from trungutt February 23, 2026 13:47
@gtardif gtardif merged commit 84fe6eb into main Feb 23, 2026
10 checks passed
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