Skip to content

ci(canary): Remove the unused updater config, update nventive packages too - #1682

Open
agneszitte wants to merge 1 commit into
mainfrom
dev/agzi/canary-cleanup
Open

agneszitte wants to merge 1 commit into
mainfrom
dev/agzi/canary-cleanup

Conversation

@agneszitte

Copy link
Copy Markdown
Member

GitHub Issue (If applicable): Related to uno-private#2074 (Uno Toolkit on Uno Platform 7.0)

PR Type

  • Build or CI related changes

What is the current behavior?

  • build/updaterconfig.json is not passed to the canary updater by any pipeline. Its Uno.Wasm.Bootstrap* entries (UpdatedVersion: 0.0.0.0) would never apply anyway.
  • The canary only updates packages published by Uno Platform / unoplatform, so it skips Uno.Core.Extensions.* and Uno.SourceGenerationTasks, which are published by nventive.

What is the new behavior?

  • build/updaterconfig.json is removed.
  • nventive is added to packageAuthor, as in Uno.Themes#1763. On canaries, Uno.SourceGenerationTasks now moves from 4.2.0 to the latest dev (4.3.0-dev.19 today). Uno.Core.Extensions.* are already the latest.
  • Nothing changes outside canaries/*: the updater steps are gated by IsCanaryBranch.

Validated locally

  • nugetupdater 1.3.1 with the new settings: Uno.Sdk.Private 7.0.0-dev.2285 → 7.0.0-dev.2423, Uno.WinUI.DevServer → 7.0.0-dev.2423, Uno.UI.RuntimeTests.Engine → 3.0.0-dev.7, Uno.XamlMerge.Task → 1.34.0-dev.3, Uno.SourceGenerationTasks → 4.3.0-dev.19.
  • With those versions, src/Uno.Toolkit-packages.slnf Release build and pack (Visual Studio MSBuild, as in Build_Packages): 0 errors, 8 packages.
  • With those versions, Uno.Toolkit.Samples.Material Release net10.0-android (the command from stage-build-android.yml): build succeeded, signed APK produced.

PR Checklist

  • Tested the changes where applicable: CI only, no app change
  • Updated the documentation as needed: not needed
  • Runtime Tests and/or UI Tests: not applicable
  • Contains NO breaking changes

Other information

Not changed here, noted for later:

  • Uno.UITest.* are published with their own package id as author, so the canary never updates them (1.1.0-dev.82 pinned, 1.1.0-dev.92 available).
  • UnoCSharpMarkupVersion is read by the updater as a package named uno.csharp.markup, which doesn't exist, so C# Markup is never updated by the canary (7.0.0-dev.33 is the latest today).
  • On the Linux jobs, NUGET_PACKAGES is inside the sources folder, so the updater also rewrites the .nuspec files of packages already in that cache (27 files in the 10-22 run). It hasn't caused a failure.

🤖 Generated with Claude Code

…s too

- Remove build/updaterconfig.json: no pipeline passes it to the canary updater, and its Uno.Wasm.Bootstrap entries (UpdatedVersion 0.0.0.0) would never apply anyway
- Add nventive to packageAuthor, so the canary also updates Uno.Core.Extensions.* and Uno.SourceGenerationTasks (published under that author), as Uno.Themes does

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@github-actions

Copy link
Copy Markdown

Azure Static Web Apps: Your stage site is ready! Visit it here: https://polite-field-01aa64f0f-1682.eastus2.azurestaticapps.net

This branch has not been deployed

No deployments
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.

1 participant