Skip to content
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

Add CheckInnerBrace and CheckPipe options to PSUseConsistentWhitespace #1092

Merged
Changes from 1 commit
Commits
Show all changes
24 commits
Select commit Hold shift + click to select a range
7e740b2
Check for 1 whitespace AFTER curly brace in new InnerCurly option
bergmeister Sep 25, 2018
cebbaad
check for one space in closing of inner brace
bergmeister Sep 25, 2018
5f444bd
add check for pipes. TODO: messages
bergmeister Sep 25, 2018
1052423
Merge branch 'development' of https://github.com/PowerShell/PSScriptA…
bergmeister Oct 3, 2018
1fbb541
Merge branch 'development' into UseConsistenWhiteSpace_InnerCurlyAndPipe
bergmeister Nov 2, 2018
60885b0
update settings files and add first set of tests for curly braces. TO…
bergmeister Nov 3, 2018
692969e
fix new line handling for innerbrace
bergmeister Nov 3, 2018
5ab67c1
fix suggested corrections and add another test case for inner brace
bergmeister Nov 3, 2018
a4f5c8d
order settings alphabetically and add tests for checkpipe
bergmeister Nov 3, 2018
b45cce5
fix test that returned 2 warnings now due to checkinnerbrace
bergmeister Nov 3, 2018
0668df9
fix innerPipe and write documentation
bergmeister Nov 3, 2018
d74412e
tweak backtick scenarios
bergmeister Nov 3, 2018
43b5882
fix 1 failing test
bergmeister Nov 3, 2018
73edc10
customise warning messages
bergmeister Nov 3, 2018
6b362d9
swap messages to be correct
bergmeister Nov 3, 2018
c872d1c
Merge branch 'development' of https://github.com/PowerShell/PSScriptA…
bergmeister Dec 19, 2018
ff3e3d1
add more test cases and fix small bug whereby the missing space befor…
bergmeister Dec 19, 2018
e4d6879
enforce whitespace also if there is more than 1 curly brace. TODO: ad…
bergmeister Dec 19, 2018
fb57292
add test cases for nested parenthesis and add validation to test help…
bergmeister Dec 19, 2018
196b112
add test case for more than 1 space inside curly braces and tidy up t…
bergmeister Dec 19, 2018
0bc8a97
Merge branch 'development' of https://github.com/PowerShell/PSScriptA…
bergmeister Jan 9, 2019
d15ba9d
Merge branch 'development' of https://github.com/PowerShell/PSScriptA…
bergmeister Jan 15, 2019
c46ae83
Merge branch 'development' of https://github.com/PowerShell/PSScriptA…
bergmeister Jan 15, 2019
59963b2
Merge branch 'development' into UseConsistenWhiteSpace_InnerCurlyAndPipe
bergmeister Mar 1, 2019
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
fix 1 failing test
bergmeister committed Nov 3, 2018

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature. The key has expired.
commit 43b5882fcbda457a415c0a6ea5c25ab6eb8cfa3e
1 change: 1 addition & 0 deletions Rules/UseConsistentWhitespace.cs
Original file line number Diff line number Diff line change
@@ -260,6 +260,7 @@ private IEnumerable<DiagnosticRecord> FindInnerBraceViolations(TokenOperations t
|| !IsPreviousTokenOnSameLine(rCurly)
|| rCurly.Previous.Value.Kind == TokenKind.LCurly
|| rCurly.Previous.Value.Kind == TokenKind.NewLine
|| rCurly.Previous.Value.Kind == TokenKind.LineContinuation
|| ((rCurly.Previous.Value.TokenFlags & TokenFlags.MemberName) == TokenFlags.MemberName))
{
continue;