Skip to content

Update AboutCmdletVerbs - #318

Merged
vexx32 merged 3 commits into
masterfrom
Comments/AboutCmdletVerbs
Oct 30, 2019
Merged

vexx32 merged 3 commits into
masterfrom
Comments/AboutCmdletVerbs

Conversation

@vexx32

@vexx32 vexx32 commented Oct 30, 2019 •

Copy link
Copy Markdown
Collaborator

PR Summary

Bit of a comment style pass and general cleanup to ensure consistency in style and phrasing across the file.

Context

Saine on the PS discord ran through this file today and brought up a couple small issues remaining in this file's comments, mainly.

Changes

  • Style pass for whitespace etc.
  • Consolidated some comments into single blocks.
  • Updated the wording of some small segments and removed some repetition.

Checklist

  • Pull Request has a meaningful title.
  • Summarised changes.
  • Pull Request is ready to merge & is not WIP.
  • Added tests / only testable interactively.
    • Make sure you add a new test if old tests do not effectively test the code changed.
  • Added documentation / opened issue to track adding documentation at a later date.

Clean up wording, consolidate comments into blocks where sensible.

Fixed some misleading examples, and removed some unnecessary repetition.
@vexx32 vexx32 added PR-Needs-Review 🔍 Let's take a closer look! Category-Koans Invoking the Great Doubt labels Oct 30, 2019
@vexx32
vexx32 requested a review from steviecoaster as a code owner October 30, 2019 02:34
@vexx32 vexx32 removed the PR-Needs-Review 🔍 Let's take a closer look! label Oct 30, 2019
@vexx32
vexx32 merged commit 6634df7 into master Oct 30, 2019
@vexx32
vexx32 deleted the Comments/AboutCmdletVerbs branch October 30, 2019 18:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Category-Koans Invoking the Great Doubt

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant