Skip to content

Add analyzer for duplicate data row - #5144

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

Youssef Fahmy (Youssef1313) merged 3 commits into
microsoft:mainfrom
Youssef1313:dup-datarow

Conversation

@Youssef1313

@Youssef1313 Youssef Fahmy (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

@Evangelink Amaury Levé (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

Amaury Levé (@Evangelink) I addressed your comments

@Youssef1313
Youssef Fahmy (Youssef1313) merged commit fa3b1ce into microsoft:main Mar 25, 2025
@Youssef1313
Youssef Fahmy (Youssef1313) deleted the dup-datarow branch March 25, 2025 07:07
stan-sz pushed a commit to stan-sz/testfx that referenced this pull request Apr 1, 2025
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