Skip to content

Unnecessary commented out code in event types? #423

Closed
@JonathanStoye

Description

@JonathanStoye
  • DOM Testing Library version: 9.4.0
  • node version: 13.4.0
  • npm (or yarn) version: yarn 1.19.1

Relevant code or config:

I just came across the events in the repo and found this:

  // error: {
  //   EventType: Event,
  //   defaultInit: {bubbles: false, cancelable: false},
  // },

Which I guess should be removed. It also exists here without comments so I am assuming this was supposed to be cleaned up at some point? Git blame says its been there for the last two years so I would say it is save to remove it.

Suggested solution:

Remove it. I am happy to help.

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions