Additional Microsoft Modules to track in PS7 Telemetry#10751
Additional Microsoft Modules to track in PS7 Telemetry#10751adityapatwardhan merged 6 commits intoPowerShell:masterfrom
Conversation
Adding additional Microsoft-owned modules to be tracked at load time in the PowerShell 7 telemetry.
Updating to include non-gallery modules queried from Microsoft Docs.
SteveL-MSFT
left a comment
There was a problem hiding this comment.
@joeyaiello needs to review this list for accuracy
|
@SteveL-MSFT does this need to get in the RC? |
|
@TravisEz13 yes |
joeyaiello
left a comment
There was a problem hiding this comment.
Whew. Finally got through this one.
We have lots of work to do in renaming Windows PowerShell modules in the Microsoft Docs, @sdwheeler, I took some notes along the way.
| "AzureRm.UsageAggregates", | ||
| "AzureRm.Websites", | ||
| "AzureRmStorageTable", | ||
| "AzureStorageTools", |
Co-Authored-By: Joey Aiello <[email protected]>
SteveL-MSFT
left a comment
There was a problem hiding this comment.
Ok from me once @joeyaiello approves
|
@SteveL-MSFT @joeyaiello Please re-review. |
joeyaiello
left a comment
There was a problem hiding this comment.
Thanks for the changes. Approving operating under the assumption that this list is not case sensitive (assuming that's what StringComparer.OrdinalIgnoreCase does, but I never can remember the exact behavior around ordinal vs. non-ordinal).
If it is case-sensitive, I have to make some more changes ASAP
|
🎉 Handy links: |
PR Summary
This adds additional Microsoft-owned modules to be tracked at load time in the PowerShell 7 telemetry.
PR Context
This allows us to better understand how PowerShell is being used to manage services and to better understand the types of PowerShell users based on which modules are loaded into PowerShell sessions.
PR Checklist
.h,.cpp,.cs,.ps1and.psm1files have the correct copyright headerWIP:or[ WIP ]to the beginning of the title (theWIPbot will keep its status check atPendingwhile the prefix is present) and remove the prefix when the PR is ready.