DotNetAnalyzers / PublicApiAnalyzer

Superseded by Microsoft.CodeAnalysis.PublicApiAnalyzers

https://www.nuget.org/packages/Microsoft.CodeAnalysis.PublicApiAnalyzers/

Other

18 stars 9 forks source link

Properly add the getter/setter of a VB auto prop to the public API. #35

Closed CyrusNajmabadi closed 5 years ago

CyrusNajmabadi commented 5 years ago

Fixes https://github.com/DotNetAnalyzers/PublicApiAnalyzer/issues/34

codecov[bot] commented 5 years ago

Codecov Report

Merging #35 into master will decrease coverage by 0.08%. The diff coverage is 73.91%.

@@            Coverage Diff             @@
##           master      #35      +/-   ##
==========================================
- Coverage   83.41%   83.32%   -0.09%     
==========================================
  Files          21       21              
  Lines        2586     2609      +23     
  Branches      193      197       +4     
==========================================
+ Hits         2157     2174      +17     
- Misses        349      353       +4     
- Partials       80       82       +2

CyrusNajmabadi commented 5 years ago

Closing in favor of https://github.com/dotnet/roslyn-analyzers/pull/2098

sharwell commented 5 years ago

@CyrusNajmabadi The implementation in roslyn-analyzers is not generally usable outside of dotnet/roslyn. Most consumers of the public API analyzer use the implementation in this repository.