On Wed, Dec 23, 2020 at 05:25:21PM +0800, Wei Jihui wrote:

load job from yaml maybe has symbol key symbol key does not has start_with function.

I think that you'd better to add changeLog for patch.

Signed-off-by: Wei Jihui weijihuiall@163.com

lib/job.rb | 7 ++++++- 1 file changed, 6 insertions(+), 1 deletion(-)

diff --git a/lib/job.rb b/lib/job.rb index 7553312e..35e0ceee 100755 --- a/lib/job.rb +++ b/lib/job.rb @@ -254,7 +254,6 @@ class Job @overrides.merge!(hash['override']){ |_key, a, _b| a} hash.delete('override') end

•    hash.delete_if { |key, _| key.start_with?('#!') }
    hash.merge!(@overrides)
    @jobs.concat(multi_args(hash)) # return [hash] or [h1,h2]
  end
  

@@ -290,6 +289,12 @@ class Job jobs_array = [] hash.each { |key, value| next unless key =~ /^\w.*|.*\w$/

•  next unless key.is_a?(String)
  

•  if key.start_with?('#! ')
  

•    hash.delete(key)
  

•    next
  

•  end
  key_array = key.split('|').map(&:strip)
  [value].flatten.each do |v|
  

-- 2.23.0

On Wed, Dec 23, 2020 at 05:25:21PM +0800, Wei Jihui wrote:

load job from yaml maybe has symbol key symbol key does not has start_with function.

How to get a symbol key, can give an example?

Thanks, RenWen

Signed-off-by: Wei Jihui weijihuiall@163.com

lib/job.rb | 7 ++++++- 1 file changed, 6 insertions(+), 1 deletion(-)

diff --git a/lib/job.rb b/lib/job.rb index 7553312e..35e0ceee 100755 --- a/lib/job.rb +++ b/lib/job.rb @@ -254,7 +254,6 @@ class Job @overrides.merge!(hash['override']){ |_key, a, _b| a} hash.delete('override') end

•    hash.delete_if { |key, _| key.start_with?('#!') }
    hash.merge!(@overrides)
    @jobs.concat(multi_args(hash)) # return [hash] or [h1,h2]
  end
  

@@ -290,6 +289,12 @@ class Job jobs_array = [] hash.each { |key, value| next unless key =~ /^\w.*|.*\w$/

•  next unless key.is_a?(String)
  

•  if key.start_with?('#! ')
  

•    hash.delete(key)
  

•    next
  

•  end
  
  key_array = key.split('|').map(&:strip)
  [value].flatten.each do |v|
  

-- 2.23.0