Follow-up from "Update feature flag to be enabled by default"

Everyone can contribute. Help move this issue forward while earning points, leveling up and collecting rewards.

The following discussions from !207741 (merged) should be addressed:

  • @dstull started a discussion: (+2 comments)

    question (non-blocking):

    Do we need to support both due to the reverse FF job?

    Meaning when the FF goes away fully we can remove this conditional?

  • @dstull started a discussion: (+2 comments)

    suggestion:

    can we instead define expected_text in the call of this shared example and remove this conditional?

    something like this

    it_behaves_like 'loads all images' do
      let(:expected_text) { "Welcome to GitLab" }
    end
Edited by 🤖 GitLab Bot 🤖