From 6ce923e1eea79469d2e5e2f3487c21afb6c73f80 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Gr=C3=A9goire=20Seux?= Date: Mon, 5 Jun 2017 09:41:37 +0200 Subject: Avoid modification on possibly frozen string MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Most users are defining option description with string literal. Starting from ruby 2.3, rubocop starts advocating to set `# frozen_string_literal: true` magic comment. Before this patch, this would raise an error related to modification of frozen string. Change-Id: Ia2561296a5855e639a4972368ab00ccbccd7cb12 Signed-off-by: Grégoire Seux --- lib/mixlib/cli.rb | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) (limited to 'lib') diff --git a/lib/mixlib/cli.rb b/lib/mixlib/cli.rb index 3c40a27..ab34b56 100644 --- a/lib/mixlib/cli.rb +++ b/lib/mixlib/cli.rb @@ -311,9 +311,10 @@ module Mixlib arguments << opt_setting[:short] if opt_setting.has_key?(:short) arguments << opt_setting[:long] if opt_setting.has_key?(:long) if opt_setting.has_key?(:description) - description = opt_setting[:description] + description = opt_setting[:description].dup description << " (required)" if opt_setting[:required] description << " (included in ['#{opt_setting[:in].join("', '")}'])" if opt_setting[:in] + opt_setting[:description] = description arguments << description end -- cgit v1.2.1