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.

Sign up for GitHub

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

Jump to bottom

Fix SA1008 not reported in cref attributes #1591

Merged
merged 2 commits into from
Oct 3, 2015
Merged

Fix SA1008 not reported in cref attributes #1591

merged 2 commits into from
Oct 3, 2015

Conversation

Noryoko
Copy link
Contributor

@Noryoko Noryoko commented Oct 2, 2015

@codecov-io
Copy link

Current coverage is 78.61%

Merging #1591 into master will increase coverage by +1.30% as of 3af5231

@@            master   #1591   diff @@
======================================
  Files          531     522     -9
  Stmts        32235   31385   -850
  Branches      8833    8837     +4
  Methods          0               
======================================
- Hit          24924   24672   -252
+ Partial       5165    5163     -2
+ Missed        2146    1550   -596

Review entire Coverage Diff as of 3af5231

Powered by Codecov. Updated on successful CI builds.

@sharwell sharwell added this to the 1.0.0 Beta 14 milestone Oct 3, 2015
@sharwell sharwell self-assigned this Oct 3, 2015
sharwell added a commit that referenced this pull request Oct 3, 2015
@sharwell
Merge pull request #1591 from Noryoko/fix-1585
3f5836c 
Fix SA1008 not reported in cref attributes
@sharwell sharwell merged commit 3f5836c into DotNetAnalyzers:master Oct 3, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@sharwell sharwell

Labels
None yet
Projects
None yet
Milestone
1.0.0 Beta 14
Development

Successfully merging this pull request may close these issues.
3 participants
@Noryoko @codecov-io @sharwell