-
Notifications
You must be signed in to change notification settings - Fork 187
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
Adjustments enabling change-specific PackageInfo #8602
Conversation
…sion on the serviceInput parameter. probably want to go to a separate script entirely? add missing line
The following pipelines have been queued for testing: |
The following pipelines have been queued for testing: |
The following pipelines have been queued for testing: |
The following pipelines have been queued for testing: |
The following pipelines have been queued for testing: |
Co-authored-by: Wes Haggard <[email protected]>
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#8602 See [eng/common workflow](https://github.com/Azure/azure-sdk-tools/blob/main/eng/common/README.md#workflow) --------- Co-authored-by: Scott Beddall (from Dev Box) <[email protected]> Co-authored-by: Scott Beddall <[email protected]> Co-authored-by: Wes Haggard <[email protected]>
* add additional argument to Save-Package-Properties to allow for usage in pull request context --------- Co-authored-by: Scott Beddall (from Dev Box) <[email protected]> Co-authored-by: Scott Beddall <[email protected]> Co-authored-by: Wes Haggard <[email protected]>
Effectively, this allows
Save-Package-Properties
to honor thediff
file.In practice it looks like this
todo:
ServiceInput
or duplicate this thing into another script entirely?On my
build
phase, this is the yml gating access to the filterEverywhere else that may need to do work against the changeset will get this
packages
artifact, then utilize it to determine targeted packagesWhat this specifically willl vary per language. For Python it's just setting
TargetingString
variable to a comma separated list of package names.EDIT: the rebase worked. got through android patch.