* final
* ytryrt
* Update .devcontainer/devcontainer.json
Co-authored-by: Victor Vazquez <victor.vazquez@microsoft.com>
Co-authored-by: Victor Vazquez <victor.vazquez@microsoft.com>
* Attempt tagging of @danieljurek in PRBody
* Attempt to assign reviewers
* Use codeowners, filter out groups, use a default owner, add comments to issue.
* Move variables to right level of config
* Set PrOwners variable
* Remove PR owners, requires admin permissions
* Escape DevOps variable to expand into string
* Add AuthToken
* Tweak portfile and codeowners to validate multiple artifact scenario
* Revert "Tweak portfile and codeowners to validate multiple artifact scenario"
This reverts commit bcc7a36119c7f399f1b079f51889d6ff0d499142.
* Review feedback
* Validate fallback logic
* Deal with empty list of owners
* Revert "Validate fallback logic"
This reverts commit ade2dff5a7d9cac9bb61fdc0f92cd50412b7c31c.
* Increment package version after release of azure-storage-blobs
* Increment package version after release of azure-storage-files-datalake
* Increment package version after release of azure-storage-files-shares
* Increment package version after release of azure-storage-queues
* Improvement: Create only necessary threads when there isn't enough data to transfer
* Update sdk/storage/azure-storage-common/inc/azure/storage/common/internal/concurrent_transfer.hpp
* CL
# Pull Request Checklist
Please leverage this checklist as a reminder to address commonly occurring feedback when submitting a pull request to make sure your PR can be reviewed quickly:
See the detailed list in the [contributing guide](https://github.com/Azure/azure-sdk-for-cpp/blob/main/CONTRIBUTING.md#pull-requests).
- [x] [C++ Guidelines](https://azure.github.io/azure-sdk/cpp_introduction.html)
- [x] Doxygen docs
- [x] Unit tests
- [x] No unwanted commits/changes
- [x] Descriptive title/description
- [x] PR is single purpose
- [x] Related issue listed
- [x] Comments in source
- [x] No typos
- [x] Update changelog
- [x] Not work-in-progress
- [x] External references or docs updated
- [x] Self review of PR done
- [] Any breaking changes?
* add support for non public scope for secret client credential
* improve function comments
* update tests with good url
* update secret url tests
* update logic to no-throw
* use class
* Update sdk/core/azure-core/inc/azure/core/http/policies/policy.hpp
Co-authored-by: Anton Kolesnyk <41349689+antkmsft@users.noreply.github.com>
* format
* fix
* remove default scope
* move patch out of Core to use package-per-client patch
* update includes
* Update sdk/keyvault/azure-security-keyvault-keys/CHANGELOG.md
Co-authored-by: Anton Kolesnyk <41349689+antkmsft@users.noreply.github.com>
* update patch
Co-authored-by: Anton Kolesnyk <41349689+antkmsft@users.noreply.github.com>
* playback tests
* two running tests
* more tests and some redaction
* formatting
* formatting
* fix bb and remove unused includes
* macos
* annoying issues
* offline client
* format
* pr comment
* reset the env to original after the client is gone.
* oops
* testing
* formats
* remove default resource for secrets, not needed
* code coverage
* build issue
* first set of changes
* another set
* add new constructor
* format files, remove pedantinc ;
* factory
* format
* expect_death not behaving as expected
Trying to reduce the noise to signal ratio and manage GitHub notifications to when I am explicitly tagged for relevant reviews and conversations, rather than during day-to-day keyvault code churn and features.
