Skip to content

Allow trailing lines in multiline comment directives #38370

Open
@JoshuaKGoldberg

Description

@JoshuaKGoldberg

TypeScript Version: 4.0.0-dev.20200507

Search Terms: multiline comment directive ts-expect-error ts-ignore

Code

const x = 0;

/* @ts-ignore
continuing comment 
 that could be multiline*/

x();

Expected behavior:

The comment looks like a directive to ignore the x();'s type checking complaint.

Actual behavior:

Comment directive parsing only looks at the last line of the comment. It should also allow the first line of the comment.

I'm in favor of not allowing any other lines (i.e. "middle" lines) to count as directives. Extending to allow characters after the @ts- has precedent from what's already allowed (we're really just adding newlines). But putting characters before the @ts- has a precedent of not working even in single-line comments.

/* Here we use a
@ts-ignore
 that is multiline*/

Playground Link: https://www.typescriptlang.org/play/?ssl=1&ssc=1&pln=28&pc=2#code/MYewdgzgLgBAHjAvDADAbgFAYPQCoYACUEAtAJYDmYIATgKYahhRlgCurFMoAtj3cxgYYUABYBDWKDYAbACYwARnRg9ZLGazq5sWOAAoAlGiA

Related Issues: Following discussion in #38228 (comment), as suggested by @sheetalkamat.

Metadata

Metadata

Labels

BugA bug in TypeScriptRescheduledThis issue was previously scheduled to an earlier milestone

Type

No type

Projects

No projects

Relationships

None yet

Development

No branches or pull requests

Issue actions