Enable frozen string for lib/gitlab/ci

Enables frozen string for the following:

* lib/gitlab/ci/*.rb
* lib/gitlab/ci/build/**/*.rb
* lib/gitlab/ci/config/**/*.rb
* lib/gitlab/ci/pipeline/**/*.rb
* lib/gitlab/ci/reports/**/*.rb

Partially addresses #47424.
This commit is contained in:
gfyoung 2018-10-25 21:12:43 -07:00
parent e997b22df5
commit 06892e88f5
90 changed files with 188 additions and 2 deletions

View File

@ -0,0 +1,5 @@
---
title: Enable frozen string for lib/gitlab/ci
merge_request:
author: gfyoung
type: performance

View File

@ -1,3 +1,5 @@
# frozen_string_literal: true
# ANSI color library
#
# Implementation per http://en.wikipedia.org/wiki/ANSI_escape_code
@ -265,7 +267,7 @@ module Gitlab
def reset_state
@offset = 0
@n_open_tags = 0
@out = ''
@out = +''
reset
end

View File

@ -1,3 +1,5 @@
# frozen_string_literal: true
module Gitlab
module Ci
module Build

View File

@ -1,3 +1,5 @@
# frozen_string_literal: true
module Gitlab
module Ci
module Build

View File

@ -1,3 +1,5 @@
# frozen_string_literal: true
require 'zlib'
require 'json'

View File

@ -1,3 +1,5 @@
# frozen_string_literal: true
module Gitlab
module Ci
module Build

View File

@ -1,3 +1,5 @@
# frozen_string_literal: true
module Gitlab
module Ci
module Build

View File

@ -1,3 +1,5 @@
# frozen_string_literal: true
module Gitlab
module Ci
module Build

View File

@ -1,3 +1,5 @@
# frozen_string_literal: true
module Gitlab
module Ci
module Build

View File

@ -1,3 +1,5 @@
# frozen_string_literal: true
module Gitlab
module Ci
module Build

View File

@ -1,3 +1,5 @@
# frozen_string_literal: true
module Gitlab
module Ci
module Build

View File

@ -1,3 +1,5 @@
# frozen_string_literal: true
module Gitlab
module Ci
module Build

View File

@ -1,3 +1,5 @@
# frozen_string_literal: true
module Gitlab
module Ci
module Build

View File

@ -1,3 +1,5 @@
# frozen_string_literal: true
module Gitlab
module Ci
module Build

View File

@ -1,3 +1,5 @@
# frozen_string_literal: true
module Gitlab
module Ci
module Build

View File

@ -1,3 +1,5 @@
# frozen_string_literal: true
module Gitlab
module Ci
module Build

View File

@ -1,3 +1,5 @@
# frozen_string_literal: true
module Gitlab
module Ci
module Build

View File

@ -1,3 +1,5 @@
# frozen_string_literal: true
module Gitlab
module Ci
module Charts

View File

@ -1,3 +1,5 @@
# frozen_string_literal: true
module Gitlab
module Ci
#

View File

@ -1,3 +1,5 @@
# frozen_string_literal: true
module Gitlab
module Ci
class Config

View File

@ -1,3 +1,5 @@
# frozen_string_literal: true
module Gitlab
module Ci
class Config

View File

@ -1,3 +1,5 @@
# frozen_string_literal: true
module Gitlab
module Ci
class Config

View File

@ -1,3 +1,5 @@
# frozen_string_literal: true
module Gitlab
module Ci
class Config

View File

@ -1,3 +1,5 @@
# frozen_string_literal: true
module Gitlab
module Ci
class Config

View File

@ -1,3 +1,5 @@
# frozen_string_literal: true
module Gitlab
module Ci
class Config

View File

@ -1,3 +1,5 @@
# frozen_string_literal: true
module Gitlab
module Ci
class Config

View File

@ -1,3 +1,5 @@
# frozen_string_literal: true
module Gitlab
module Ci
class Config

View File

@ -1,3 +1,5 @@
# frozen_string_literal: true
module Gitlab
module Ci
class Config

View File

@ -1,3 +1,5 @@
# frozen_string_literal: true
module Gitlab
module Ci
class Config

View File

@ -1,3 +1,5 @@
# frozen_string_literal: true
module Gitlab
module Ci
class Config

View File

@ -1,3 +1,5 @@
# frozen_string_literal: true
module Gitlab
module Ci
class Config

View File

@ -1,3 +1,5 @@
# frozen_string_literal: true
module Gitlab
module Ci
class Config

View File

@ -1,3 +1,5 @@
# frozen_string_literal: true
module Gitlab
module Ci
class Config

View File

@ -1,3 +1,5 @@
# frozen_string_literal: true
module Gitlab
module Ci
class Config

View File

@ -1,3 +1,5 @@
# frozen_string_literal: true
module Gitlab
module Ci
class Config

View File

@ -1,3 +1,5 @@
# frozen_string_literal: true
module Gitlab
module Ci
class Config

View File

@ -1,3 +1,5 @@
# frozen_string_literal: true
module Gitlab
module Ci
class Config

View File

@ -1,3 +1,5 @@
# frozen_string_literal: true
module Gitlab
module Ci
class Config

View File

@ -1,3 +1,5 @@
# frozen_string_literal: true
module Gitlab
module Ci
class Config

View File

@ -1,3 +1,5 @@
# frozen_string_literal: true
module Gitlab
module Ci
class Config

View File

@ -1,3 +1,5 @@
# frozen_string_literal: true
module Gitlab
module Ci
class Config

View File

@ -1,3 +1,5 @@
# frozen_string_literal: true
module Gitlab
module Ci
class Config

View File

@ -1,3 +1,5 @@
# frozen_string_literal: true
module Gitlab
module Ci
class Config

View File

@ -1,3 +1,5 @@
# frozen_string_literal: true
module Gitlab
module Ci
class Config

View File

@ -1,3 +1,5 @@
# frozen_string_literal: true
module Gitlab
module Ci
class Config

View File

@ -1,3 +1,5 @@
# frozen_string_literal: true
module Gitlab
module Ci
class Config

View File

@ -1,3 +1,5 @@
# frozen_string_literal: true
module Gitlab
module Ci
class Config

View File

@ -1,3 +1,5 @@
# frozen_string_literal: true
module Gitlab
module Ci
class Config

View File

@ -1,3 +1,5 @@
# frozen_string_literal: true
module Gitlab
module Ci
class Config

View File

@ -1,3 +1,5 @@
# frozen_string_literal: true
module Gitlab
module Ci
class Config

View File

@ -1,3 +1,5 @@
# frozen_string_literal: true
module Gitlab
module Ci
class Config

View File

@ -1,3 +1,5 @@
# frozen_string_literal: true
module Gitlab
module Ci
class CronParser

View File

@ -1,9 +1,13 @@
# frozen_string_literal: true
module Gitlab
module Ci::MaskSecret
class << self
def mask!(value, token)
return value unless value.present? && token.present?
# We assume 'value' must be mutable, given
# that frozen string is enabled.
value.gsub!(token, 'x' * token.length)
value
end

View File

@ -1,3 +1,5 @@
# frozen_string_literal: true
module Gitlab
module Ci
module Model

View File

@ -1,3 +1,5 @@
# frozen_string_literal: true
module Gitlab
module Ci
module Pipeline

View File

@ -1,3 +1,5 @@
# frozen_string_literal: true
module Gitlab
module Ci
module Pipeline

View File

@ -1,4 +1,7 @@
module Gitlab # rubocop:disable Naming/FileName
# rubocop:disable Naming/FileName
# frozen_string_literal: true
module Gitlab
module Ci
module Pipeline
module Chain

View File

@ -1,3 +1,5 @@
# frozen_string_literal: true
module Gitlab
module Ci
module Pipeline

View File

@ -1,3 +1,5 @@
# frozen_string_literal: true
module Gitlab
module Ci
module Pipeline

View File

@ -1,3 +1,5 @@
# frozen_string_literal: true
module Gitlab
module Ci
module Pipeline

View File

@ -1,3 +1,5 @@
# frozen_string_literal: true
module Gitlab
module Ci
module Pipeline

View File

@ -1,3 +1,5 @@
# frozen_string_literal: true
module Gitlab
module Ci
module Pipeline

View File

@ -1,3 +1,5 @@
# frozen_string_literal: true
module Gitlab
module Ci
module Pipeline

View File

@ -1,3 +1,5 @@
# frozen_string_literal: true
module Gitlab
module Ci
module Pipeline

View File

@ -1,3 +1,5 @@
# frozen_string_literal: true
module Gitlab
module Ci
module Pipeline

View File

@ -1,3 +1,5 @@
# frozen_string_literal: true
module Gitlab
module Ci
module Pipeline

View File

@ -1,3 +1,5 @@
# frozen_string_literal: true
module Gitlab
module Ci
module Pipeline

View File

@ -1,3 +1,5 @@
# frozen_string_literal: true
module Gitlab
module Ci
module Pipeline

View File

@ -1,3 +1,5 @@
# frozen_string_literal: true
module Gitlab
module Ci
module Pipeline

View File

@ -1,3 +1,5 @@
# frozen_string_literal: true
module Gitlab
module Ci
module Pipeline

View File

@ -1,3 +1,5 @@
# frozen_string_literal: true
module Gitlab
module Ci
module Pipeline

View File

@ -1,3 +1,5 @@
# frozen_string_literal: true
module Gitlab
module Ci
module Pipeline

View File

@ -1,3 +1,5 @@
# frozen_string_literal: true
module Gitlab
module Ci
module Pipeline

View File

@ -1,3 +1,5 @@
# frozen_string_literal: true
module Gitlab
module Ci
module Pipeline

View File

@ -1,3 +1,5 @@
# frozen_string_literal: true
module Gitlab
module Ci
module Pipeline

View File

@ -1,3 +1,5 @@
# frozen_string_literal: true
module Gitlab
module Ci
module Pipeline

View File

@ -1,3 +1,5 @@
# frozen_string_literal: true
module Gitlab
module Ci
module Pipeline

View File

@ -1,3 +1,5 @@
# frozen_string_literal: true
module Gitlab
module Ci
module Pipeline

View File

@ -1,3 +1,5 @@
# frozen_string_literal: true
module Gitlab
module Ci
module Pipeline

View File

@ -1,3 +1,5 @@
# frozen_string_literal: true
module Gitlab
module Ci
module Pipeline

View File

@ -1,3 +1,5 @@
# frozen_string_literal: true
module Gitlab
module Ci
module Pipeline

View File

@ -1,3 +1,5 @@
# frozen_string_literal: true
module Gitlab
module Ci
module Pipeline

View File

@ -1,3 +1,5 @@
# frozen_string_literal: true
module Gitlab
module Ci
module Pipeline

View File

@ -1,3 +1,5 @@
# frozen_string_literal: true
module Gitlab
module Ci
module Reports

View File

@ -1,3 +1,5 @@
# frozen_string_literal: true
module Gitlab
module Ci
module Reports

View File

@ -1,3 +1,5 @@
# frozen_string_literal: true
module Gitlab
module Ci
module Reports

View File

@ -1,3 +1,5 @@
# frozen_string_literal: true
module Gitlab
module Ci
module Reports

View File

@ -1,3 +1,5 @@
# frozen_string_literal: true
module Gitlab
module Ci
module Reports

View File

@ -1,3 +1,5 @@
# frozen_string_literal: true
module Gitlab
module Ci
class Trace

View File

@ -1,3 +1,5 @@
# frozen_string_literal: true
module Gitlab
module Ci
class YamlProcessor