Skip to content

Cleanup highlight_js_worker feature flag

Jacques Erasmus requested to merge 432706-cleanup-highlight_js_worker-ff into master

What does this MR do and why?

The highlight_js_worker feature flag has been enabled by default for a while now. This MR cleans up the feature flag and renames related components:

Why rename components as part of the cleanup?

With the highlight_js_worker enabled we used the source_viewer_new.vue component, since this is now our primary component for viewing source code, it becomes source_viewer.vue and the old one is deleted.

Summary:

  1. Remove all references of highlight_js_worker
  2. Delete source_viewer.vue
  3. Rename source_viewer_new.vue to source_viewer.vue
  4. Update related specs

MR acceptance checklist

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

Screenshots or screen recordings

No visual changes are expected

How to set up and validate locally

  1. Open a source code file via the repository files browser
  2. The source code syntax should be highlighted

Related to #432706 (closed)

Edited by Jacques Erasmus

Merge request reports