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

MA0015 supports primary constructors and records #559

Merged
merged 1 commit into from
Jul 12, 2023
Merged

Conversation

meziantou
Copy link
Owner

Fix #558

@meziantou meziantou changed the title MA0015 supports primary constructors MA0015 supports primary constructors and records Jul 12, 2023
@codecov
Copy link

codecov bot commented Jul 12, 2023

Codecov Report

Merging #559 (284fddb) into main (ba0aad1) will increase coverage by 0.05%.
The diff coverage is 98.93%.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #559      +/-   ##
==========================================
+ Coverage   90.08%   90.14%   +0.05%     
==========================================
  Files         177      177              
  Lines       13030    12986      -44     
==========================================
- Hits        11738    11706      -32     
+ Misses       1292     1280      -12     
Impacted Files Coverage Δ
...eziantou.Analyzer/Internals/OperationExtensions.cs 88.66% <98.55%> (+8.41%) ⬆️
...c/Meziantou.Analyzer/Internals/SymbolExtensions.cs 91.83% <100.00%> (+0.92%) ⬆️
...umentExceptionShouldSpecifyArgumentNameAnalyzer.cs 87.20% <100.00%> (-0.06%) ⬇️
...s/UseAnOverloadThatHasCancellationTokenAnalyzer.cs 89.80% <100.00%> (-0.37%) ⬇️
...tou.Analyzer/Rules/UseDateTimeUnixEpochAnalyzer.cs 98.61% <100.00%> (ø)

@meziantou meziantou changed the title MA0015 supports primary constructors and records MA0015 supports primary constructors Jul 12, 2023
@meziantou meziantou changed the title MA0015 supports primary constructors MA0015 supports primary constructors and records Jul 12, 2023
@meziantou meziantou force-pushed the issues/558 branch 2 times, most recently from e113090 to 90afab4 Compare July 12, 2023 23:02
@meziantou meziantou marked this pull request as ready for review July 12, 2023 23:22
@meziantou meziantou enabled auto-merge (squash) July 12, 2023 23:22
@meziantou meziantou disabled auto-merge July 12, 2023 23:23
@meziantou meziantou enabled auto-merge (squash) July 12, 2023 23:23
@meziantou meziantou merged commit 2251423 into main Jul 12, 2023
21 checks passed
@meziantou meziantou deleted the issues/558 branch July 12, 2023 23:26
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.

MA0015 false positive for record properties
1 participant