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

Update naming rules to support records #3209

Merged
merged 1 commit into from
Sep 27, 2020

Conversation

sharwell
Copy link
Member

@sharwell sharwell commented Sep 26, 2020

Closes #3168
Fixes #3181

Comment on lines +47 to +51
// /0/Test0.cs(2,15): warning SA1300: Element 'r' should begin with an uppercase letter
Diagnostic().WithLocation(0).WithArguments("r"),

// /0/Test0.cs(2,15): warning SA1300: Element 'r' should begin with an uppercase letter
Diagnostic().WithLocation(0).WithArguments("r"),
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

📝 This is a bug in the analyzer driver

@sharwell sharwell marked this pull request as draft September 26, 2020 22:44
@sharwell sharwell marked this pull request as ready for review September 26, 2020 23:22
@codecov
Copy link

codecov bot commented Sep 27, 2020

Codecov Report

Merging #3209 into master will increase coverage by 0.00%.
The diff coverage is 100.00%.

@@           Coverage Diff            @@
##           master    #3209    +/-   ##
========================================
  Coverage   97.12%   97.13%            
========================================
  Files         839      841     +2     
  Lines      104842   104946   +104     
  Branches     3413     3416     +3     
========================================
+ Hits       101828   101937   +109     
+ Misses       2099     2095     -4     
+ Partials      915      914     -1     

@sharwell sharwell merged commit 6399f49 into DotNetAnalyzers:master Sep 27, 2020
@sharwell sharwell deleted the positional-record branch September 27, 2020 01:24
@sharwell sharwell added this to the 1.2-beta.next milestone Sep 27, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
1 participant