Skip to content

Trailing comma after deleting two adjacent nodes in a list #24201

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
ghost opened this issue May 17, 2018 · 0 comments
Closed

Trailing comma after deleting two adjacent nodes in a list #24201

ghost opened this issue May 17, 2018 · 0 comments
Labels
Bug A bug in TypeScript Domain: Formatter The issue relates to the built-in formatter Fixed A PR has been merged for this issue

Comments

@ghost
Copy link

ghost commented May 17, 2018

TypeScript Version: 2.9.0-dev.20180516

Code

/// <reference path='fourslash.ts' />

////function f(t, u, v) { return t; }

verify.codeFixAll({
    fixId: "unusedIdentifier_delete",
    fixAllDescription: "Delete all unused declarations",
    newFileContent:
`function f(t) { return t; }`,
});

Expected behavior:

Test passes.

Actual behavior:

     Error: verifyFileContent failed:
Expected:
function f(t) { return t; }
Actual:
function f(t, ) { return t; }
@ghost ghost added Bug A bug in TypeScript Domain: Formatter The issue relates to the built-in formatter labels May 17, 2018
@ghost ghost self-assigned this May 17, 2018
@mhegazy mhegazy added this to the TypeScript 2.9.1 milestone May 17, 2018
@ghost ghost added the Fixed A PR has been merged for this issue label May 17, 2018
@ghost ghost closed this as completed in #24221 May 17, 2018
@microsoft microsoft locked and limited conversation to collaborators Jul 31, 2018
This issue was closed.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Bug A bug in TypeScript Domain: Formatter The issue relates to the built-in formatter Fixed A PR has been merged for this issue
Projects
None yet
Development

No branches or pull requests

1 participant