Skip to content

Remove Legacy Web IDE frontend

Enrique Alcántararequested to merge
537160-ealcantara/remove-legacy-web-ide-frontend into master

Issue: Remove legacy Web IDE frontend source code (#537160 - closed)

What does this MR do and why?

It removes the source code of the legacy Web IDE that isn't used by other features in the GitLab application. For example:

  • It removes Vue components and Vuex store implementations.
  • It removes CSS stylesheets.
  • It removes custom ESLint configuration and rule violations exceptions.

NOTICE This Merge Request doesn't remove any code in app/assets/javascripts/ide that is used by other modules in app/assets/javascripts. For example, we noticed that the modules app/assets/javascripts/editor and app/assets/javascripts/diffs depend on some utilities defined in the legacy Web IDE. This MR doesn't attempt to refactor these dependencies with the goal of keeping its scope under control. We will defer this refactoring to follow-up Merge Requests.

We removed the entry point to the Legacy Web IDE and the feature flag that controlled access its access !188427 (merged). This Merge Request doesn't have user-facing consequences because the code removed is not referenced by any production feature.

References

Screenshots or screen recordings

This Merge Request doesn't introduce user-facing changes.

How to set up and validate locally

We are removing code that is not referenced by any features in the GitLab application.

MR acceptance checklist

Evaluate this MR against the MR acceptance checklist. It helps you analyze changes to reduce risks in quality, performance, reliability, security, and maintainability.

Edited by Enrique Alcántara

Merge request reports