Skip to content

Conversation

@DavidMMaxwell
Copy link
Contributor

@DavidMMaxwell DavidMMaxwell commented Mar 9, 2020

Updated example 2 to use Get-AzSqlInstanceDatabase instead of Get-AzSqlDatabase.

Description

Checklist

  • I have read the Submitting Changes section of CONTRIBUTING.md
  • The title of the PR is clear and informative
  • The appropriate ChangeLog.md file(s) has been updated:
    • For any service, the ChangeLog.md file can be found at src/{{SERVICE}}/{{SERVICE}}/ChangeLog.md
    • A snippet outlining the change(s) made in the PR should be written under the ## Upcoming Release header -- no new version header should be added
  • The PR does not introduce breaking changes
  • If applicable, the changes made in the PR have proper test coverage
  • For public API changes to cmdlets:
    • a cmdlet design review was approved for the changes in this repository (Microsoft internal only)
    • the markdown help files have been regenerated using the commands listed here

shamidt and others added 30 commits February 14, 2020 16:30
add Sql to Network.sln
…pendency

Add system.buffers dependency to az.datalakestore
[IoT] Add cmdlet to get device connection string
[IoT Hub] Get module connection string cmdlet
Editions in the ChangeLog of CosmosDB
Fix of example in Set-AzDiskKeyEncryptionKey and Set-AzDiskDiskEncryptionKey
@adxsdkps
Copy link
Collaborator

adxsdkps commented Mar 9, 2020

Can one of the admins verify this patch?

@isra-fel isra-fel self-assigned this Mar 11, 2020
@isra-fel
Copy link
Member

Thank you for the contribution!
@jaredmoo can you help review the example? Thanks

@isra-fel isra-fel requested a review from jaredmoo March 11, 2020 08:07
@isra-fel
Copy link
Member

isra-fel commented Mar 16, 2020

Hello @adeal could you help review this community contribution to docs? Thanks

Copy link
Member

@isra-fel isra-fel left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sorry to make your PR a mess... we just force pushed master so it needed some cleanup.
Thanks for contribution again!

@isra-fel
Copy link
Member

/azp run

@azure-pipelines
Copy link
Contributor

Azure Pipelines successfully started running 3 pipeline(s).

@isra-fel isra-fel merged commit b0e0bac into Azure:master Apr 14, 2020
@DavidMMaxwell
Copy link
Contributor Author

DavidMMaxwell commented Apr 14, 2020 via email

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.