Skip to content

[RUN AS-IF-FOSS] Update gitlab-styles to 5.1.0

Doug Stull requested to merge 277422-update-gitlab-styles-5.1.0 into master

What does this MR do?

  • Updates gitlab-styles to 5.1.0
  • Removes LineBreakAroundConditionalBlock local in favor of upstreamed version in gitlab-styles
  • Fixes empty line issue with let_it_be provided with gitlab-styles update. Note, this linting rule was already agreed upon with !36742 (merged), but had to disable again when adding in the fix for let_it_be. This re-enables it after fixing the exceptions.

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

Related to #277422, #263244

Edited by Doug Stull

Merge request reports