Fixes Issue #340 #339 #335 #330

[kungfoome]

Pull Request (PR) description

This Pull Request (PR) fixes the following issues

• Fixes xADUser: This resource is missing comment-based help #340
• Fixes xADOrganizationalUnit: This resource is missing comment-based help #339
• Fixes xADDomain: This resource is missing comment-based help #335
• Fixes xADRecycleBin: This resource is missing comment-based help #330

Task list

• Added an entry under the Unreleased section in the CHANGELOG.md.
  Entry should say what was changed, and how that affects users (if applicable).
• Resource documentation added/updated in resource directory README.md.
• Resource parameter descriptions added/updated in schema.mof
  and comment-based help.
• Comment-based help added/updated.
• Localization strings added/updated in all localization files as appropriate.
• Examples appropriately added/updated.
• Unit tests added/updated. See DSC Resource Testing Guidelines.
• Integration tests added/updated (where possible). See DSC Resource Testing Guidelines.
• New/changed code adheres to DSC Resource Style Guidelines and Best Practices.

---

This change is 

[johlju]

Thank you for this! Just tiny comments.

Reviewed 5 of 5 files at r1.
Reviewable status: all files reviewed, 2 unresolved discussions (waiting on @kungfu71186)

---

CHANGELOG.md, line 75 at r1 (raw file):

modules

From the user perspective this should be: resources

---

DSCResources/MSFT_xADDomain/MSFT_xADDomain.psm1, line 49 at r1 (raw file):

<# Get-TargetResource

Should just be <#. Throughout all comment-based help.

[codecov-io]

Codecov Report

Merging #440 into dev will not change coverage.
The diff coverage is 100%.

@@         Coverage Diff         @@
##            dev   #440   +/-   ##
===================================
  Coverage    96%    96%           
===================================
  Files        20     20           
  Lines      2705   2705           
  Branches     10     10           
===================================
  Hits       2608   2608           
  Misses       87     87           
  Partials     10     10

[kungfoome]

Reviewable status: all files reviewed, 2 unresolved discussions (waiting on @johlju)

---

CHANGELOG.md, line 75 at r1 (raw file):

Previously, johlju (Johan Ljunggren) wrote…

modules

From the user perspective this should be: resources

Done.

---

DSCResources/MSFT_xADDomain/MSFT_xADDomain.psm1, line 49 at r1 (raw file):

Previously, johlju (Johan Ljunggren) wrote…

<# Get-TargetResource

Should just be <#. Throughout all comment-based help.

Done.

[johlju]

Reviewed 5 of 5 files at r2.
Reviewable status: complete! all files reviewed, all discussions resolved