fix(semantic): don't report abstract method with implementation twice

[ulrichstark]

The match above the deleted line already reports depending on the kind of method.

[Copilot]

Pull Request Overview

This pull request fixes a duplicate error reporting issue in the TypeScript semantic checker where abstract methods with implementation were being reported twice.

• Removes redundant error reporting for abstract methods with implementation
• Fixes double error messages by relying on existing match statement handling

---

_{Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.}

[codspeed-hq]

CodSpeed Instrumentation Performance Report

Merging #14193 will not alter performance

_{Comparing ulrichstark:don't-report-abstract-method-with-implementation-twice (6e9370d) with main (5b74a81)¹}

Summary

✅ 37 untouched

Footnotes

1. No successful run was found on main (5dcc64c) during the generation of this report, so 5b74a81 was used instead as the comparison base. There might be some changes unrelated to this pull request in this report. ↩

[graphite-app]

Merge activity

• Sep 28, 4:18 AM UTC: @ulrichstark we removed the merge queue label because we could not find a Graphite account associated with your GitHub profile.

You must have a Graphite account in order to use the merge queue. Create an account and try again using this link