-
Notifications
You must be signed in to change notification settings - Fork 500
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
Update community_snippets.md #1395
Conversation
Added the Mail snippet
Renamed the snippet
docs/community_snippets.md
Outdated
#### Snippet | ||
|
||
```json | ||
"Send-MailMessage": { |
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 recommend naming the snippet something other than the command name. This can get confusing when you think you're autocompleting the command name but then get this snippet. We have been using a naming convention in the extension where "examples" start with ex-
. Perhaps you call this snippet ex-send-mail-message
?
Leave a comment |
Snippet renamed to ex-Send-MailMessage which stands for "example of Send-MailMessage"
Snippet renamed to ex-Send-MailMessage |
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.
LGTM. Thanks for your contribution!
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.
LGTM 🎉 Thanks!
* Update community_snippets.md Added the Mail snippet * Update community_snippets.md Renamed the snippet * Update community_snippets.md Snippet renamed to ex-Send-MailMessage which stands for "example of Send-MailMessage"
PR Summary
Added the Send-MailMessage snippet
PR Checklist