Skip to content

Fixes Monaco library loading on merge requests

Phil Hughes requested to merge ph/diffsMonacoJS into master

What does this MR do?

This stops the Monaco library from being loaded on merge requests. On GitLab.com this file is more than 600kb and is never used, it only gets included because we are using a import that also imports the monaco editor.

Screenshots

Does this MR meet the acceptance criteria?

Conformity

Availability and Testing

Security

If this MR contains changes to processing or storing of credentials or tokens, authorization and authentication methods and other items described in the security review guidelines:

  • Label as security and @ mention @gitlab-com/gl-security/appsec
  • The MR includes necessary changes to maintain consistency between UI, API, email, or other methods
  • Security reports checked/validated by a reviewer from the AppSec team

Merge request reports