-
Notifications
You must be signed in to change notification settings - Fork 183
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Changes required for sdk-repo based emitter pipelines #9147
Conversation
The following pipelines have been queued for testing: |
eng/common/pipelines/templates/stages/archetype-typespec-emitter.yml
Outdated
Show resolved
Hide resolved
eng/common/pipelines/templates/stages/archetype-typespec-emitter.yml
Outdated
Show resolved
Hide resolved
eng/common/pipelines/templates/stages/archetype-typespec-emitter.yml
Outdated
Show resolved
Hide resolved
eng/common/pipelines/templates/stages/archetype-typespec-emitter.yml
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
A few comments but nothing blocking.
eng/common/pipelines/templates/stages/archetype-typespec-emitter.yml
Outdated
Show resolved
Hide resolved
eng/common/pipelines/templates/stages/archetype-typespec-emitter.yml
Outdated
Show resolved
Hide resolved
The following pipelines have been queued for testing: |
The following pipelines have been queued for testing: |
The following pipelines have been queued for testing: |
Sync eng/common directory with azure-sdk-tools for PR Azure/azure-sdk-tools#9147 See [eng/common workflow](https://github.com/Azure/azure-sdk-tools/blob/main/eng/common/README.md#workflow) --------- Co-authored-by: Patrick Hallisey <[email protected]>
Changes required by pr Azure/azure-sdk-for-net#46511
Chages tested in pipeline run internal/4224817
archetype-typespec-emitter.yml
:create-authenticated-npmrc.yml
:archetype-typespec-emitter.yml
CommandInvocation-Helpers.ps1
:Invoke-LoggedCommand
to CmdLet to support -ErrorAction parameterNew-RegenerateMatrix.ps1
:Get-${Language}-DirectoriesForGeneration
and only filter packages with it inNew-RegenerateMatrix
if there's no language override.New-EmitterPackageJson.ps1
:I wanted to simplify the way branded packages can force emitter peer pinning in emitter-package.json without having to restate all of the unbranded emitter's peer dependencies. By adding a custom section to package.json we can list the devDependencies that should be forwarded to emitter-package.json for peer pinning.
instead of:
we can: