Skip to content
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

Sync eng/common directory with azure-sdk-tools for PR 1546 #17948

Merged
merged 1 commit into from
Apr 9, 2021

Conversation

azure-sdk
Copy link
Collaborator

Sync eng/common directory with azure-sdk-tools for PR Azure/azure-sdk-tools#1546 See eng/common workflow

Initializing the BaseName with a ServiceDirectory that contains
a "/", because it is a multiple level path, causes the BaseName
initialization code to fail because it doesn't support the
validation pattern.

In all the known cases we already pass the ResourceGroupName
explicitly so don't need to set the BaseName so we can
skip the initialization in those cases.
@azure-sdk azure-sdk added Central-EngSys This issue is owned by the Engineering System team. EngSys This issue is impacting the engineering system. labels Apr 9, 2021
@azure-sdk azure-sdk merged commit d2af6af into master Apr 9, 2021
@azure-sdk azure-sdk deleted the sync-eng/common-FixBaseNameInitialization-1546 branch April 9, 2021 20:33
azure-sdk pushed a commit to azure-sdk/azure-sdk-for-python that referenced this pull request Apr 24, 2022
[Hub Generated] Review request for Microsoft.Devices to add version stable/2022-02-05 (Azure#17948)

* Adds base for updating Microsoft.Devices from version stable/2021-10-15 to version 2022-02-05

* Updates readme

* Updates API version in new specs and examples

* definition changes

* example changes

* no comments

* swagger modelval fix #1

* general validation fix #1

* revert readonly change (breaking change)

* added locatio pvtEp example

* misplaced location param

* location in PvtEConn response

* header addition

* headers #2

* example change with location sans spec

* header nesting change

* header under headers

* header example
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Central-EngSys This issue is owned by the Engineering System team. EngSys This issue is impacting the engineering system.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants