Skip to content

[appservice ext] Fix ci-build.ps1 bugs: version path, ldflags, typo#6812

Merged
vhvb1989 merged 2 commits intoappservice-extfrom
copilot/sub-pr-6811
Feb 20, 2026
Merged

[appservice ext] Fix ci-build.ps1 bugs: version path, ldflags, typo#6812
vhvb1989 merged 2 commits intoappservice-extfrom
copilot/sub-pr-6811

Conversation

Copy link
Contributor

Copilot AI commented Feb 20, 2026

Three bugs in cli/azd/extensions/azure.appservice/ci-build.ps1 that would break the release build pipeline.

Changes

  • Version path: $PSScriptRoot/../version.txt$PSScriptRoot/version.txtversion.txt is co-located with the script, not in the parent directory
  • ldflags: Was referencing wrong module (azureaiagent) and non-existent variables (Commit, BuildDate); corrected to match the actual module name and the sole variable in internal/version/version.go:
    # Before
    $ldFlag = "-ldflags=-s -w -X 'azureaiagent/internal/version.Version=$Version' -X 'azureaiagent/internal/version.Commit=$SourceVersion' -X 'azureaiagent/internal/version.BuildDate=$(Get-Date -Format o)' "
    
    # After
    $ldFlag = "-ldflags=-s -w -X 'azureappservice/internal/version.Version=$Version'"
  • Typo: variaiblevariable in comment

✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

…typo

Co-authored-by: vhvb1989 <24213737+vhvb1989@users.noreply.github.com>
Copilot AI changed the title [WIP] Add pipeline definition for azure.appservice extension [appservice ext] Fix ci-build.ps1 bugs: version path, ldflags, typo Feb 20, 2026
Copilot AI requested a review from vhvb1989 February 20, 2026 04:02
@vhvb1989 vhvb1989 marked this pull request as ready for review February 20, 2026 04:08
@vhvb1989 vhvb1989 merged commit e4638dd into appservice-ext Feb 20, 2026
1 check 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.

2 participants