PowerShell / Community-Blog

Submissions for posts to the PowerShell Community Blog -https://devblogs.microsoft.com/powershell-community

MIT License

108 stars 47 forks source link

Fixes #41 - keeping it classy #43

Closed sdwheeler closed 3 years ago

sdwheeler commented 3 years ago

PR Summary

Fixes #41 - keeping it classy

PR Checklist

Do you want this post to be published on a specific date?: Y/N [Specify date]:
[x] I have read the contributors guide and followed the style and process guidelines
[x] PR has a meaningful title
[x] Includes content related an open issue
- see Closing issues using keywords.
[x] This PR is ready to merge and is not Work in Progress

CLAassistant commented 3 years ago

All committers have signed the CLA.

doctordns commented 3 years ago

Looks ok to me but sad to see those semantic sentences removed. OH well... :-)

sdwheeler commented 3 years ago

@doctordns But our style guide is not true "semantic sentences" but 100 characters limit. The goal is readability and easy diffs when viewing the source, not semantic blocking. Besides, by the definition on https://sembr.org/, you should break sentences in to clauses or phrases, not use full sentences.