-
Notifications
You must be signed in to change notification settings - Fork 52
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
DiskAccessPath Add Disk Subsystem Refresh before set - Fixes #121 #209
Conversation
Codecov Report
@@ Coverage Diff @@
## dev #209 +/- ##
===================================
+ Coverage 95% 95% +<1%
===================================
Files 7 7
Lines 958 978 +20
===================================
+ Hits 917 937 +20
Misses 41 41 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Reviewed 5 of 5 files at r1.
Reviewable status: all files reviewed, 3 unresolved discussions (waiting on @PlagueHO)
DSCResources/MSFT_DiskAccessPath/MSFT_DiskAccessPath.psm1, line 761 at r1 (raw file):
<# .SYNOPSIS Check access path is found in PSDrives.
The text should be indented one more step. Throughout.
DSCResources/MSFT_DiskAccessPath/MSFT_DiskAccessPath.psm1, line 771 at r1 (raw file):
wihtout
typo
DSCResources/MSFT_DiskAccessPath/MSFT_DiskAccessPath.psm1, line 810 at r1 (raw file):
$accessPathDrive
Maybe add a verbose message saying if the access path was found or not? 🤔
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Reviewable status: all files reviewed, 5 unresolved discussions (waiting on @PlagueHO)
DSCResources/MSFT_DiskAccessPath/MSFT_DiskAccessPath.psm1, line 797 at r1 (raw file):
$localizedData
$script:localizedData
DSCResources/MSFT_DiskAccessPath/MSFT_DiskAccessPath.psm1, line 806 at r1 (raw file):
$localizedData
$script:localizedData
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks @johlju - should be good to go!
Reviewable status: 2 of 9 files reviewed, 4 unresolved discussions (waiting on @johlju)
DSCResources/MSFT_DiskAccessPath/MSFT_DiskAccessPath.psm1, line 761 at r1 (raw file):
Previously, johlju (Johan Ljunggren) wrote…
The text should be indented one more step. Throughout.
Done. And also in all the other files 😁
DSCResources/MSFT_DiskAccessPath/MSFT_DiskAccessPath.psm1, line 771 at r1 (raw file):
Previously, johlju (Johan Ljunggren) wrote…
wihtout
typo
Done.
DSCResources/MSFT_DiskAccessPath/MSFT_DiskAccessPath.psm1, line 797 at r1 (raw file):
Previously, johlju (Johan Ljunggren) wrote…
$localizedData
$script:localizedData
Done.
DSCResources/MSFT_DiskAccessPath/MSFT_DiskAccessPath.psm1, line 806 at r1 (raw file):
Previously, johlju (Johan Ljunggren) wrote…
$localizedData
$script:localizedData
Done.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Reviewed 7 of 7 files at r2.
Reviewable status: complete! all files reviewed, all discussions resolved
Thanks @johlju !!! |
Pull Request (PR) description
Set-TargetResource to prevent errors occuring when the disk access
path is already assigned - See Issue 121
This PR also supercedes #152, #126
This Pull Request (PR) fixes the following issues
Task list
Entry should say what was changed, and how that affects users (if applicable).
and comment-based help.
@johlju - would you mind reviewing this when you get a chance?
This change is