Skip to content

SA1115 will now allow comments between parameters#1981

Merged
sharwell merged 1 commit intoDotNetAnalyzers:masterfrom
vweijsters:fix-1921
Dec 30, 2015
Merged

SA1115 will now allow comments between parameters#1981
sharwell merged 1 commit intoDotNetAnalyzers:masterfrom
vweijsters:fix-1921

Conversation

@vweijsters
Copy link
Copy Markdown
Contributor

Fixes #1921

@codecov-io
Copy link
Copy Markdown

Current coverage is 93.28%

Merging #1981 into master will increase coverage by +13.85% as of efe2855

@@            master   #1981   diff @@
======================================
  Files          559     561      +2
  Stmts        35237   35477    +240
  Branches      9751    2276   -7475
  Methods                           
======================================
+ Hit          27992   33096   +5104
+ Partial       5694     750   -4944
- Missed        1551    1631     +80

Review entire Coverage Diff as of efe2855

Powered by Codecov. Updated on successful CI builds.

@Noryoko
Copy link
Copy Markdown
Contributor

Noryoko commented Dec 22, 2015

👍

@sharwell sharwell added this to the 1.1.0 Beta 1 milestone Dec 22, 2015
@sharwell sharwell added the fixed label Dec 28, 2015
@sharwell sharwell merged commit 21f7ecf into DotNetAnalyzers:master Dec 30, 2015
sharwell added a commit that referenced this pull request Dec 30, 2015
SA1115 will now allow comments between parameters
@vweijsters vweijsters deleted the fix-1921 branch December 30, 2015 10:00
@sharwell sharwell removed the fixed label Nov 23, 2016
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