Skip to content

Add analyzer for duplicate data row#5144

Merged
Youssef1313 merged 3 commits into
microsoft:mainfrom
Youssef1313:dup-datarow
Mar 25, 2025
Merged

Add analyzer for duplicate data row#5144
Youssef1313 merged 3 commits into
microsoft:mainfrom
Youssef1313:dup-datarow

Conversation

@Youssef1313

@Youssef1313 Youssef1313 commented Feb 26, 2025

Copy link
Copy Markdown
Member

Fixes #5085

@microsoft-github-policy-service microsoft-github-policy-service Bot added the area/analyzers MSTest.Analyzers Roslyn analyzers and code fixes. label Mar 1, 2025
@Youssef1313 Youssef1313 marked this pull request as ready for review March 11, 2025 08:43
@Youssef1313 Youssef1313 requested a review from nohwnd March 11, 2025 08:43

@Evangelink Evangelink left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Please add a test where you have 2 pairs of duplicate attributes on the same method

Comment thread src/Analyzers/MSTest.Analyzers/DuplicateDataRowAnalyzer.cs Outdated
Comment thread src/Analyzers/MSTest.Analyzers/Resources.resx Outdated
Comment thread src/Analyzers/MSTest.Analyzers/Resources.resx Outdated
@Youssef1313

Copy link
Copy Markdown
Member Author

@Evangelink I addressed your comments

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/analyzers MSTest.Analyzers Roslyn analyzers and code fixes.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Analyzer suggestion: Duplicate DataRow

2 participants