* Change logging functions to use Write-Host
Change logging helpers to always write to host and either set color or use devops/gh formatting.
We do not want to use Write-Error or Write-Warning directly because they can stop the script or not depending on preferences which makes it difficult to ensure local runs of scripts work the same as in the pipelines. So, we should never depend on these logging commands to cause a script to stop execution.
* Clean up error handling in CommandInvocation-Helpers
Removed legacy error handling for command failures.
* Fix error logging for command execution
* Add option to skip exiting
---------
Co-authored-by: Wes Haggard <weshaggard@users.noreply.github.com>
Co-authored-by: Wes Haggard <Wes.Haggard@microsoft.com>
Fails if you don't have `bicep` installed but do have `az bicep` and was unnecessary since the check below it did it anyway.
Co-authored-by: Heath Stewart <heaths@microsoft.com>
* Updated validate pkg template to use packageInfo
* Fixed typo
* Fixed the right variable to use
* output debug log
* Fixed errors in expression evaluation
* removed debug code
* Fixed an issue in pipeline
* Updated condition for variable setting step
* Join paths of the script path
* Use join-path
* return from the function rather than exit
---------
Co-authored-by: ray chen <raychen@microsoft.com>
* Mark TypeSpec upgrade PR as draft when generation fails
Co-authored-by: JoshLove-msft <54595583+JoshLove-msft@users.noreply.github.com>
* Add condition to run Create_PR job even when Generate fails
Co-authored-by: JoshLove-msft <54595583+JoshLove-msft@users.noreply.github.com>
* Include emitter name in PR title
Co-authored-by: JoshLove-msft <54595583+JoshLove-msft@users.noreply.github.com>
* Use emitterIdentifier variable for PR title instead of recalculating
Co-authored-by: JoshLove-msft <54595583+JoshLove-msft@users.noreply.github.com>
* Include scope in emitterIdentifier for branch and PR names
Co-authored-by: JoshLove-msft <54595583+JoshLove-msft@users.noreply.github.com>
* Read emitter name from package.json instead of extracting from path
Co-authored-by: JoshLove-msft <54595583+JoshLove-msft@users.noreply.github.com>
* Resolve emitterPackagePath to absolute path before reading
Co-authored-by: JoshLove-msft <54595583+JoshLove-msft@users.noreply.github.com>
* Append package.json to emitterPackagePath directory
Co-authored-by: JoshLove-msft <54595583+JoshLove-msft@users.noreply.github.com>
* Add log statement for fallback to package path
Co-authored-by: JoshLove-msft <54595583+JoshLove-msft@users.noreply.github.com>
---------
Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: JoshLove-msft <54595583+JoshLove-msft@users.noreply.github.com>
* Update create-apireview template to use packageInfo
* Use new signature for FindArtifactForApiReviewFn
* Used new signature of FindArtifactForApiReviewFn
---------
Co-authored-by: ray chen <raychen@microsoft.com>
* Added 'ArtifactsJson' parameter
* Made script work for non-java
* Reduced duplicated parsing
* Fixed type error
---------
Co-authored-by: ray chen <raychen@microsoft.com>
* Revise tsp-client usage instructions in README
Updated usage instructions to use 'npm exec --prefix' for running tsp-client commands. The reason is `tsp-client` needs to resolve the input parameters based on the root folder of SDK repository.
* Update README.md to use brace expansion for variables
* Update README.md
---------
Co-authored-by: Ray Chen <raychen@microsoft.com>
* Pass packageInfo parameter conditionally to Find-Language-Artifacts-For-Apireview
* Use new parameter
---------
Co-authored-by: ray chen <raychen@microsoft.com>
* Set pr creator to empty when it's bot
* Removed bot accounts while setting the reviewers and assignees
* Allowed copilot as reviewer
---------
Co-authored-by: ray chen <raychen@microsoft.com>
* bump the test-proxy version
* bump proxy to one that includes our multipart/formdata fix
---------
Co-authored-by: Scott Beddall (from Dev Box) <scbedd@microsoft.com>
* pin tsp-client
* Added a script to wrap the call to tsp-client
* Used npm i to improve per
* Removed script and added readme.md
* Update eng/common/tsp-client/README.md
Co-authored-by: Mike Harder <mharder@microsoft.com>
* Update eng/common/tsp-client/README.md
Co-authored-by: Mike Harder <mharder@microsoft.com>
* Use npm ci for installation
* Added dependabot to readme
* Update eng/common/tsp-client/README.md
Co-authored-by: Mike Harder <mharder@microsoft.com>
* Removed 'prefix' option approach from usage
* Fixed format issue
---------
Co-authored-by: catalinaperalta <caperal@microsoft.com>
Co-authored-by: ray chen <raychen@microsoft.com>
Co-authored-by: Mike Harder <mharder@microsoft.com>
* snake case
* change file name to .instructions.md
* Revert "change file name to .instructions.md"
This reverts commit c4bb65dc03258a45f74590edff6ccbf889d9e559.
---------
Co-authored-by: Juan Ospina <70209456+jeo02@users.noreply.github.com>
