Skip to content

Remove glfm-verify CI job and related tests

Himanshu Kapoor requested to merge himkp-remove-glfm-verify-ci-job into master

What does this MR do and why?

Remove glfm-verify CI job and related tests

Removes tests related to markdown snapshots and the glfm-verify job. This is the first step to completely remove all tests related to GLFM snapshot specs.

Previous work related to this removal: !160563 (merged)

We can also close #402321 (closed) once this MR is merged.

This is a part 1 of split of !160852. Since that MR has 53k lines removed, it was only logical to split it. 😅

Why are we removing GLFM specs?

In summary, GLFM specs, while a good initiative, no longer serve their purpose. They are slow, redundant and unnecessarily complicated. We already have sufficient coverage at the unit and integration test levels for our Markdown parser and serializers:

  • Parser specs: link
  • Serializer specs: link
  • Markdown API feature specs: link
  • Rich text editor feature specs: link

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.

Edited by Himanshu Kapoor

Merge request reports