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

Fixed issue in SA1500CodeFixProvider with brace at the end of the sou… #2106

Merged
merged 1 commit into from
Mar 17, 2016

Conversation

vweijsters
Copy link
Contributor

…rce file

Fixes #2105

@sharwell sharwell added this to the 1.1.0 Beta 1 milestone Mar 17, 2016
@sharwell sharwell self-assigned this Mar 17, 2016
@codecov-io
Copy link

Current coverage is 93.35%

Merging #2106 into master will decrease coverage by -0.01% as of eb57323

@@            master   #2106   diff @@
======================================
  Files          566     566       
  Stmts        35937   35946     +9
  Branches      2315    2315       
  Methods                          
======================================
+ Hit          33551   33557     +6
  Partial        761     761       
- Missed        1625    1628     +3

Review entire Coverage Diff as of eb57323

Powered by Codecov. Updated on successful CI builds.

sharwell added a commit that referenced this pull request Mar 17, 2016
Fixed issue in SA1500CodeFixProvider with brace at the end of the sou…
@sharwell sharwell merged commit 42e577f into DotNetAnalyzers:master Mar 17, 2016
@sharwell sharwell deleted the fix-2105 branch March 17, 2016 22:28
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.

3 participants