Skip to content

Fix preview CI missing preview version suffix - #61

Merged
MikeAlhayek merged 2 commits into
mainfrom
copilot/fix-preview-ci-deployment
Apr 30, 2026
Merged

Fix preview CI missing preview version suffix#61
MikeAlhayek merged 2 commits into
mainfrom
copilot/fix-preview-ci-deployment

Conversation

Copilot AI commented Apr 30, 2026

Copy link
Copy Markdown
Contributor

preview_ci.yml was not passing -p:VersionSuffix=preview to dotnet build or dotnet pack, causing packages to publish as 1.0.0 instead of 1.0.0-preview-X.

Changes

  • Set build number: switched from $(( $GITHUB_RUN_NUMBER )) to $GITHUB_RUN_NUMBER so the raw run number is used (e.g. 1.0.0-preview-42)
  • dotnet build: added /p:VersionSuffix=preview so built assemblies carry the suffix
  • dotnet pack: added /p:VersionSuffix=preview so packed .nupkg files get the suffix

Directory.Build.props already has the wiring to append BuildNumber when VersionSuffix is non-empty, so these additions are sufficient to restore 1.0.0-preview-X-style versions.

Agent-Logs-Url: https://github.com/CrestApps/CrestApps.Core/sessions/497da273-73da-4ba7-8dc2-c42b57f7d884

Co-authored-by: MikeAlhayek <24724371+MikeAlhayek@users.noreply.github.com>
@MikeAlhayek

Copy link
Copy Markdown
Member

@copilot it does not need the padding of 4 digits. it should only follow 1.0.0-preview-X where x is the run number.

Agent-Logs-Url: https://github.com/CrestApps/CrestApps.Core/sessions/71514d18-45da-43a9-bd5c-68ec765f1904

Co-authored-by: MikeAlhayek <24724371+MikeAlhayek@users.noreply.github.com>

Copilot AI commented Apr 30, 2026

Copy link
Copy Markdown
Contributor Author

@copilot it does not need the padding of 4 digits. it should only follow 1.0.0-preview-X where x is the run number.

Done in 26f8b4a — the build number now uses the raw run number, producing versions like 1.0.0-preview-42.

Copilot AI requested a review from MikeAlhayek April 30, 2026 19:43
@MikeAlhayek
MikeAlhayek marked this pull request as ready for review April 30, 2026 19:47
@MikeAlhayek
MikeAlhayek merged commit b1624fc into main Apr 30, 2026
8 checks passed
@MikeAlhayek
MikeAlhayek deleted the copilot/fix-preview-ci-deployment branch April 30, 2026 19:47
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.

2 participants