[Feature Flag] Remove merged YAML [RUN ALL RSPEC] [RUN AS-IF-FOSS]

app/assets/javascripts/pipeline_editor/components/pipeline_editor_tabs.vue

@@ -134,7 +134,6 @@ export default {
       <ci-lint v-else :is-valid="isValid" :ci-config="ciConfigData" />
     </editor-tab>
     <editor-tab
-      v-if="glFeatures.ciConfigMergedTab"
       class="gl-mb-3"
       :empty-message="$options.i18n.empty.merge"
       :keep-component-mounted="false"

app/controllers/projects/ci/pipeline_editor_controller.rb

@@ -3,7 +3,6 @@
 class Projects::Ci::PipelineEditorController < Projects::ApplicationController
   before_action :check_can_collaborate!
   before_action do
-    push_frontend_feature_flag(:ci_config_merged_tab, @project, default_enabled: :yaml)
     push_frontend_feature_flag(:pipeline_editor_empty_state_action, @project, default_enabled: :yaml)
     push_frontend_feature_flag(:pipeline_editor_branch_switcher, @project, default_enabled: :yaml)
     push_frontend_feature_flag(:pipeline_editor_drawer, @project, default_enabled: :yaml)

changelogs/unreleased/301103-feature-flag-rollout-merged-yaml-tab-in-pa-home.yml

+---
+title: Removed feature flag for Pipeline editor merged YAML tab
+merge_request: 60659
+author:
+type: other

config/feature_flags/development/ci_config_merged_tab.yml

----
-name: ci_config_merged_tab
-introduced_by_url: https://gitlab.com/gitlab-org/gitlab/-/merge_requests/53299
-rollout_issue_url: https://gitlab.com/gitlab-org/gitlab/-/issues/301103
-milestone: '13.9'
-type: development
-group: group::pipeline authoring
-default_enabled: true

doc/ci/pipeline_editor/index.md

@@ -73,8 +73,7 @@ each job depends only on the previous stage being completed successfully.
 ## View expanded configuration
 
 > - [Introduced](https://gitlab.com/gitlab-org/gitlab/-/issues/246801) in GitLab 13.9.
-> - It is [deployed behind a feature flag](../../user/feature_flags.md), enabled by default.
-> - To use it in GitLab self-managed instances, ask a GitLab administrator to [enable it](#enable-or-disable-expanded-configuration). **(FREE SELF)**
+> - [Feature flag removed](https://gitlab.com/gitlab-org/gitlab/-/issues/301103) in GitLab 13.12.
 
 To view the fully expanded CI/CD configuration as one combined file, go to the
 pipeline editor's **View merged YAML** tab. This tab displays an expanded configuration
@@ -85,25 +84,6 @@ where:
   [extended configuration merged into the job](../yaml/README.md#merge-details).
 - YAML anchors are [replaced with the linked configuration](../yaml/README.md#anchors).
 
-### Enable or disable expanded configuration **(FREE SELF)**
-
-Expanded CI/CD configuration is under development and not ready for production use.
-It is deployed behind a feature flag that is **enabled by default**.
-[GitLab administrators with access to the GitLab Rails console](../../administration/feature_flags.md)
-can opt to enable it.
-
-To disable it:
-
-```ruby
-Feature.disable(:ci_config_merged_tab)
-```
-
-To enable it:
-
-```ruby
-Feature.enable(:ci_config_merged_tab)
-```
-
 ## Commit changes to CI configuration
 
 The commit form appears at the bottom of each tab in the editor so you can commit

spec/frontend/pipeline_editor/components/pipeline_editor_tabs_spec.js

@@ -20,11 +20,6 @@ describe('Pipeline editor tabs component', () => {
   const MockTextEditor = {
     template: '<div />',
   };
-  const mockProvide = {
-    glFeatures: {
-      ciConfigMergedTab: true,
-    },
-  };
 
   const createComponent = ({
     props = {},
@@ -43,7 +38,7 @@ describe('Pipeline editor tabs component', () => {
           appStatus,
         };
       },
-      provide: { ...mockProvide, ...provide },
+      provide: { ...provide },
       stubs: {
         TextEditor: MockTextEditor,
         EditorTab,
@@ -81,7 +76,6 @@ describe('Pipeline editor tabs component', () => {
   });
 
   describe('visualization tab', () => {
-    describe('with feature flag on', () => {
     describe('while loading', () => {
       beforeEach(() => {
         createComponent({ appStatus: EDITOR_APP_STATUS_LOADING });
@@ -103,7 +97,6 @@ describe('Pipeline editor tabs component', () => {
       });
     });
   });
-  });
 
   describe('lint tab', () => {
     describe('while loading', () => {
@@ -132,7 +125,6 @@ describe('Pipeline editor tabs component', () => {
   });
 
   describe('merged tab', () => {
-    describe('with feature flag on', () => {
     describe('while loading', () => {
       beforeEach(() => {
         createComponent({ appStatus: EDITOR_APP_STATUS_LOADING });
@@ -153,7 +145,7 @@ describe('Pipeline editor tabs component', () => {
         expect(findAlert().text()).toBe(wrapper.vm.$options.errorTexts.loadMergedYaml);
       });
 
-        it('does not render the `meged_preview` component', () => {
+      it('does not render the `merged_preview` component', () => {
         expect(findMergedPreview().exists()).toBe(false);
       });
     });
@@ -169,17 +161,6 @@ describe('Pipeline editor tabs component', () => {
       });
     });
   });
-    describe('with feature flag off', () => {
-      beforeEach(() => {
-        createComponent({ provide: { glFeatures: { ciConfigMergedTab: false } } });
-      });
-
-      it('does not display the merged tab', () => {
-        expect(findMergedTab().exists()).toBe(false);
-        expect(findMergedPreview().exists()).toBe(false);
-      });
-    });
-  });
 
   describe('show tab content based on status', () => {
     it.each`