Skip to content

Copy edit variables comments in Secure CI templates

What does this MR do?

In a recent template updated, I offered a copy edit to the variables note. See !60546 (comment 568742587)

After seeing !62161 (merged), I realized we could improve the text even further, and apply it as a standard across all Secure template.

This MR only touches code comments in the templates and docs. No configuration is changed. In the docs, the only change is to make sure we always say CI/CD variables, which is the standard.

The updates:

  • use a better link to variables documentation
  • clearer text
  • better links to each template's variables documentation.

Screenshots (strongly suggested)

Does this MR meet the acceptance criteria?

Conformity

Availability and Testing

Security

Does this MR contain changes to processing or storing of credentials or tokens, authorization and authentication methods or other items described in the security review guidelines? If not, then delete this Security section.

  • 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 Marcel Amirault

Merge request reports