-
-
Notifications
You must be signed in to change notification settings - Fork 174
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
Refactor AboutCmdletVerbs #378
Conversation
6999d65
to
e4c262a
Compare
6b4fb98
to
6757e53
Compare
6757e53
to
37df6d5
Compare
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.
I only marked a few spots here. There are a couple more, but was getting repetitive
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.
I would like to suggest to use different examples than Office 365, not everyone will have to interact with Exchange and will never see these Cmdlets, or at least give an eample additional exmaple cmdlet to the Exchange calendar cmdlets. Focus is on th verbs so the noun should be agnostic, I still think it may help readers in their learning process.
Otherwise I spotted some typos here and there.
@Maamue I agree the office 365 isn't the most applicable example for everyone. The "generic" example, so to speak, is the Is there a better / more applicable secondary example than office 365 cmdlets for this purpose, do you think? Perhaps we'd be better off varying the examples rather than using examples all from the same module? |
058a941
to
569a6cd
Compare
PR Summary
Quick pass over AboutCmdletVerbs to tidy it up a bit, get it in line with the rest of the introductory topics.
Changes
Checklist