Skip to content

Disable Code Suggestion for ipynb diffs

What does this MR do and why?

We added a cleaner diffs for jupyter notebooks (#341825 (closed)), but when a code suggestion is applied to the notebook, it breaks the file (#345075 (closed)). As mitigation, we are disabling code suggestions for notebooks.

Note: This MR doesn't provide a changelog because it is part of a feature that is being deployed in this version

Screenshots or screen recordings

Before After
image image

These are strongly recommended to assist reviewers and reduce the time to merge your change.

How to set up and validate locally

  1. Enable the feature flag:
  2. Open the rails consoles (rails c)
  3. Feature.enable(:jupyter_clean_diffs)
  4. Create a mirror of this https://gitlab.com/gitlab-org/incubation-engineering/mlops/ipynb-test-projects/diff-sample into your GDK instance
  5. Make sure the feature is enabled and create an MR from any branch into main (eg svg-latex)
  6. Create a suggestion
  7. Verify that the suggestions can't be applied

MR acceptance checklist

This checklist encourages us to confirm any changes have been analyzed to reduce risks in quality, performance, reliability, security, and maintainability.

Resolves #345075 (closed)

Edited by Pedro Moreira da Silva

Merge request reports

Loading