Skip to content

Update README.md - #1402

Merged
sergei (vors) merged 1 commit into
PowerShell:masterfrom
alexandair:patch-5
Jul 20, 2016
Merged

Update README.md#1402
sergei (vors) merged 1 commit into
PowerShell:masterfrom
alexandair:patch-5

Conversation

@alexandair

Copy link
Copy Markdown
Contributor

Added newlines following one-sentence-per-line principle.
Changed "Open PowerShell" to "PowerShell".
Fixed typos.

Added newlines following one-sentence-per-line principle.
Changed "Open PowerShell" to "PowerShell".
Fixed typos.
@joeyaiello

Copy link
Copy Markdown
Contributor

LGTM other than the wonkiness of extra newlines in between list items. Unless sergei (@vors) for Andy Jordan (@andschwa) has a reason for that, I think there generally shouldn't be extra blank newlines in a bulleted list (though, to be clear, there should still be semantic line breaks after each sentence).

@andyleejordan

Copy link
Copy Markdown
Member

This should be rebased onto master, where we have the necessary changes to enable Travis CI builds from forks (yay!).

@jianyunt

Copy link
Copy Markdown
Contributor

sergei (@vors), can you Merge please.

@vors
sergei (vors) merged commit 9ad725f into PowerShell:master Jul 20, 2016
@vors
sergei (vors) deleted the patch-5 branch July 20, 2016 22:48
Thatgfsj (Thatgfsj) pushed a commit to Thatgfsj/PowerShell that referenced this pull request Aug 6, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants