Remove obsolete checks

webpro-nl · Jan 15, 2025 · 50a777f · 50a777f
1 parent 201ee51
commit 50a777f
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/packages/knip/fixtures/class-members/members.ts b/packages/knip/fixtures/class-members/members.ts
@@ -54,6 +54,9 @@ export class MyClass {
   public set unusedSetter(value: string) {
     console.log(value);
   }
+
+  /** @public */
+  public set unusedTaggedSetter(value: string) {}
 }
 
 MyClass.displayName = 'My Class';
diff --git a/packages/knip/src/ProjectPrincipal.ts b/packages/knip/src/ProjectPrincipal.ts
@@ -364,16 +364,13 @@ export class ProjectPrincipal {
     }
 
     return members.filter(member => {
-      if (member.jsDocTags.has(PUBLIC_TAG)) return false;
       const referencedSymbols = this.findReferences?.(filePath, member.pos) ?? [];
       const refs = referencedSymbols.flatMap(refs => refs.references).filter(ref => !ref.isDefinition);
       return refs.length === 0;
     });
   }
 
   public hasExternalReferences(filePath: string, exportedItem: Export) {
-    if (exportedItem.jsDocTags.has(PUBLIC_TAG)) return false;
-
     if (!this.findReferences) {
       const languageService = ts.createLanguageService(this.backend.languageServiceHost, ts.createDocumentRegistry());
       this.findReferences = timerify(languageService.findReferences);