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

Added checks of ArrowExpressionClausSyntax to ensure correct spacing #1783

Merged
merged 1 commit into from
Nov 17, 2015
Merged

Conversation

bexxx
Copy link
Collaborator

@bexxx bexxx commented Nov 16, 2015

Added checks of ArrowExpressionClausSyntax to ensure correct spacing + unit test.
Fixes already worked with the new diagnostic locations (yeah!).

This fixes issue #1764.

…+ unit test.

Fixes already worked with the new diagnostic locations (yeah!).
@codecov-io
Copy link

Current coverage is 79.43%

Merging #1783 into master will decrease coverage by -0.02% as of 038d6d3

@@            master   #1783   diff @@
======================================
  Files          548     548       
  Stmts        32875   32918    +43
  Branches      9111    9116     +5
  Methods                          
======================================
+ Hit          26120   26147    +27
  Partial       5300    5300       
- Missed        1455    1471    +16

Review entire Coverage Diff as of 038d6d3

Powered by Codecov. Updated on successful CI builds.

@vweijsters
Copy link
Contributor

👍 Looks good to me

@sharwell sharwell added this to the 1.0.0 Beta 16 milestone Nov 17, 2015
sharwell added a commit that referenced this pull request Nov 17, 2015
Added checks of ArrowExpressionClausSyntax to ensure correct spacing
@sharwell sharwell merged commit 3d4e192 into DotNetAnalyzers:master Nov 17, 2015
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.

4 participants