Skip to content

Update dma-sku-recommend-sql-db.md#5613

Merged
ktoliver merged 3 commits intoMicrosoftDocs:livefrom
Fronzoli:patch-1
Nov 19, 2020
Merged

Update dma-sku-recommend-sql-db.md#5613
ktoliver merged 3 commits intoMicrosoftDocs:livefrom
Fronzoli:patch-1

Conversation

@Fronzoli
Copy link
Copy Markdown
Contributor

Both changes made in Data Collection Script section

Removed the quotes around the connection string in the sample invocation for the data collection script, those quotes stop the script from connecting to the database

Added a note about the WMI cmdlets needing to be updated to the CIM cmdlets to run the data collection script in Powershell 6 or 7.

Removed the quotes around the connection string in the sample invocation for the data collection script, those quotes stop the script from connecting to the database

Added a note about the WMI cmdlets needing to be updated to the CIM cmdlets to run the data collection script in Powershell 6 or 7.
@PRMerger15
Copy link
Copy Markdown
Contributor

@Fronzoli : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@rothja
Copy link
Copy Markdown
Collaborator

rothja commented Nov 3, 2020

Reassigning to @neetusingh-ms for review. Neetu, can you please review the changes on the File Changes tab and see if you approve this suggestion from @Fronzoli? Thank you! #reassign:neetusingh-ms

@ktoliver
Copy link
Copy Markdown
Contributor

Hi @neetusingh-ms Could you review this pull request per the request from @rothja? Thanks!

@PRMerger17
Copy link
Copy Markdown
Contributor

@ktoliver : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@PRMerger17
Copy link
Copy Markdown
Contributor

@ktoliver : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@ktoliver ktoliver merged commit 86fda22 into MicrosoftDocs:live Nov 19, 2020
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.

8 participants