Skip to content

Assertion failure when combining insertNodeAfter with deleteNode #23793

Closed
Closed
Assertion failure when combining insertNodeAfter with deleteNode#23793
Labels
BugA bug in TypeScriptDomain: FormatterThe issue relates to the built-in formatterFixedA PR has been merged for this issue
Milestone
TypeScript 2.9

Description

@ghost
ghost
opened on Apr 30, 2018

TypeScript Version: master

Code

See comment in moveToNewFile_js.ts (as of #23726).
We are moving const y away, and also inserting a node after the previous statement. We can't simply use replaceNode since we don't know that these two changes will be related.

Activity

  1. ghost added
    BugA bug in TypeScript
    on Apr 30, 2018
  2. ghost added
    Domain: FormatterThe issue relates to the built-in formatter
    on Apr 30, 2018
  3. Mohamed Hegazy (mhegazy)
    assigned
    ghost
    on Apr 30, 2018
  4. Mohamed Hegazy (mhegazy)
    added this to the TypeScript 2.9 milestone on Apr 30, 2018
  5. ghost added
    FixedA PR has been merged for this issue
    on May 2, 2018
  6. Mohamed Hegazy (mhegazy)
    closed this as completedon May 10, 2018
  7. microsoft
    locked and limited conversation to collaborators on Jul 31, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    BugA bug in TypeScriptDomain: FormatterThe issue relates to the built-in formatterFixedA PR has been merged for this issue

    Type

    No type

    Projects

    No projects

      Milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions