-
Notifications
You must be signed in to change notification settings - Fork 31
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
Fixes #82 #83
Fixes #82 #83
Conversation
Codecov Report
@@ Coverage Diff @@
## dev #83 +/- ##
=================================
+ Coverage 93% 98% +4%
=================================
Files 7 6 -1
Lines 443 517 +74
=================================
+ Hits 416 508 +92
+ Misses 27 9 -18 |
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 18 of 18 files at r1.
Reviewable status: all files reviewed, 1 unresolved discussion (waiting on @PlagueHO)
Modules/ActiveDirectoryCSDsc.Common/ActiveDirectoryCSDsc.Common.psm1, line 483 at r1 (raw file):
$script:localizedData = Get-LocalizedData -ResourceName 'ActiveDirectoryCSDsc.Common' -ScriptRoot $PSScriptRoot
Maybe have this at the end just before the Export-ModuleMember so it is less hidden :)
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.
All done! Thanks @johlju
Reviewable status: 17 of 18 files reviewed, 1 unresolved discussion (waiting on @johlju)
Modules/ActiveDirectoryCSDsc.Common/ActiveDirectoryCSDsc.Common.psm1, line 483 at r1 (raw file):
Previously, johlju (Johan Ljunggren) wrote…
$script:localizedData = Get-LocalizedData -ResourceName 'ActiveDirectoryCSDsc.Common' -ScriptRoot $PSScriptRoot
Maybe have this at the end just before the Export-ModuleMember so it is less hidden :)
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 1 of 1 files at r2.
Reviewable status: complete! all files reviewed, all discussions resolved
Pull Request (PR) description
ActiveDirectoryCSDsc.ResourceHelper
module functions intoActiveDirectoryCSDsc.Common
module and renamed toActiveDirectoryCSDsc.CommonHelper
module.
fixes Issue #82.
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 when you have time?
This change is