Skip to content

fix(ci/scripts): correct csharp pack powershell script to set arguments correctly#4304

Merged
lidavidm merged 1 commit intoapache:mainfrom
birschick-bq:dev/birschick-bq/fix-csharp-pack-ps-script
May 7, 2026
Merged

fix(ci/scripts): correct csharp pack powershell script to set arguments correctly#4304
lidavidm merged 1 commit intoapache:mainfrom
birschick-bq:dev/birschick-bq/fix-csharp-pack-ps-script

Conversation

@birschick-bq
Copy link
Copy Markdown
Contributor

Corrects the command arguments to dotnet pack for the --no-build flag.

  • change the argument's value from $true to '' (empty string) as it is a flag with no argument value.
  • add a command to return the working folder back to the original it was called from.

@birschick-bq birschick-bq requested review from kou and lidavidm as code owners May 7, 2026 02:03
@lidavidm lidavidm merged commit 886cfa5 into apache:main May 7, 2026
82 of 88 checks passed
@birschick-bq
Copy link
Copy Markdown
Contributor Author

@lidavidm - Thanks!

lidavidm pushed a commit that referenced this pull request May 9, 2026
Correct bug introduced in PR
#4304
- Instead uses the Push-Location / Pop-Location pair
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