Skip to content

Show merged version of ci config [RUN AS-IF-FOSS] [RUN ALL RSPEC]

Frédéric Caplette requested to merge show-merged-version-of-ci-config into master

What does this MR do?

This adds a tab that shows the mergedYaml value of a CI config inside the pipeline Editor section. It also adds some additional logic to hide the commit form when in the merged tab to avoid confusion for the user about what their file looks like when committing.

Behind a disabled feature flag: ci_config_merged_tab

Screenshots (strongly suggested)

Before After
Screen_Shot_2021-02-09_at_1.45.35_PM Screen_Shot_2021-02-09_at_1.41.06_PM

Without errors

Screen_Shot_2021-02-09_at_1.41.13_PM

With errors

Screen_Shot_2021-02-09_at_1.41.27_PM

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
Edited by 🤖 GitLab Bot 🤖

Merge request reports