Skip to content

Use npm's xterm

Clement Ho requested to merge xterm-npm into master

What does this MR do?

  • Remove glTerminal from gl scope
  • Remove xterm vendor directory in favor of npm versioned xterm
  • Remove vendor directory polyfill for TextEncoder and TextDecoder in favor of npm versioned polyfill

Are there points in the code the reviewer needs to double check?

Shouldn't be

Why was this MR needed?

Reduce vendor files in favor of npm package control

Screenshots (if relevant)

Does this MR meet the acceptance criteria?

What are the relevant issue numbers?

https://gitlab.com/gitlab-org/gitlab-ce/issues/39072

Edited by Clement Ho

Merge request reports