Delete DuplicateAttributeCheck

Dlang-UPB · Feb 20, 2024 · 2ef327d · 2ef327d
1 parent 2bf728f
commit 2ef327d
Show file tree

Hide file tree

Showing 3 changed files with 2 additions and 270 deletions.
diff --git a/changelog/dscanner.duplicate-attribute-check.dd b/changelog/dscanner.duplicate-attribute-check.dd
@@ -0,0 +1,2 @@
+Remove the check for duplicate attributes (@property, @safe, @trusted, @system, pure, nothrow).
+This check is no longer necessary since having duplicated attributes is now a compiler error.
diff --git a/src/dscanner/analysis/duplicate_attribute.d b/src/dscanner/analysis/duplicate_attribute.d
diff --git a/src/dscanner/analysis/run.d b/src/dscanner/analysis/run.d
@@ -840,10 +840,6 @@ private BaseAnalyzer[] getAnalyzersForModuleAndConfig(string fileName,
 		checks ~= new UnmodifiedFinder(args.setSkipTests(
 		analysisConfig.could_be_immutable_check == Check.skipTests && !ut));
 
-	if (moduleName.shouldRun!DuplicateAttributeCheck(analysisConfig))
-		checks ~= new DuplicateAttributeCheck(args.setSkipTests(
-		analysisConfig.duplicate_attribute == Check.skipTests && !ut));
-
 	if (moduleName.shouldRun!FunctionAttributeCheck(analysisConfig))
 		checks ~= new FunctionAttributeCheck(args.setSkipTests(
 		analysisConfig.function_attribute_check == Check.skipTests && !ut));