Skip to content

feat: enable checking @ts-expect-error directives in multi-line comments#696

Merged
mrazauskas merged 4 commits intomainfrom
multi-line-ts-expect-error
Feb 27, 2026
Merged

feat: enable checking @ts-expect-error directives in multi-line comments#696
mrazauskas merged 4 commits intomainfrom
multi-line-ts-expect-error

Conversation

@mrazauskas
Copy link
Member

@mrazauskas mrazauskas commented Feb 27, 2026 •
edited
Loading

This PR enables checking error message suppressed by @ts-expect-error directives in multi-line comments. This is useful in contexts where single line comments are not supported. For example, JSX files:

function ProfileCard(props: { userName: string }) {
  return <>{props.userName}</>;
}

<>
  {/* @ts-expect-error  Type 'boolean' is not assignable to type 'string'. */}
  <ProfileCard userName={true} />
</>;

@codacy-production
Copy link

codacy-production bot commented Feb 27, 2026

Coverage summary from Codacy

See diff coverage on Codacy

Coverage variation Diff coverage
✅ +0.06% (target: -1.00%) ✅ 100.00% (target: 85.00%)
Coverage variation details
Coverable lines Covered lines Coverage
Common ancestor commit (3039cde) 7476 7356 98.39%
Head commit (39f8631) 7400 (-76) 7286 (-70) 98.46% (+0.06%)

Coverage variation is the difference between the coverage for the head and common ancestor commits of the pull request branch: <coverage of head commit> - <coverage of common ancestor commit>

Diff coverage details
Coverable lines Covered lines Diff coverage
Pull request (#696) 2 2 100.00%

Diff coverage is the percentage of lines that are covered by tests out of the coverable lines that the pull request added or modified: <covered lines added or modified>/<coverable lines added or modified> * 100%

See your quality gate settings    Change summary preferences

@mrazauskas mrazauskas merged commit cacb565 into main Feb 27, 2026
22 checks passed
@mrazauskas mrazauskas deleted the multi-line-ts-expect-error branch February 27, 2026 11:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@mrazauskas