-
Notifications
You must be signed in to change notification settings - Fork 198
Use the updated config in regenerate step #10765
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
Conversation
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.
Pull Request Overview
This PR removes the sparse-checkout step in the regenerate stage and switches to a full repository checkout.
- Removes the custom sparse-checkout template
- Adds a
checkout: self
step withfetchDepth: 0
to pull all files
Comments suppressed due to low confidence (2)
eng/common/pipelines/templates/archetype-typespec-emitter.yml:310
- [nitpick] Clarify the inline comment by specifying the exact file path, e.g.,
# including changes in archetype-typespec-emitter/package.json
to make the reason more explicit.
- checkout: self # don't use sparse checkout here as we need the emitter package json changes
eng/common/pipelines/templates/archetype-typespec-emitter.yml:311
- Using
fetchDepth: 0
fetches the entire history, which can slow down the pipeline. ConsiderfetchDepth: 1
if only the latest commit is needed.
- fetchDepth: 0
a41e23c
to
85daf43
Compare
85daf43
to
84c749f
Compare
84c749f
to
f8f3cc6
Compare
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#10765 See [eng/common workflow](https://github.com/Azure/azure-sdk-tools/blob/main/eng/common/README.md#workflow) Co-authored-by: jolov <[email protected]>
Fixes Azure/azure-sdk-for-net#50291