Skip to content

Fix a broken .include?(Regexp) call

Rémy Coutable requested to merge 345153-job-failed-1759784214 into master

What does this MR do and why?

This fixes a silly .include?(Regexp) call introduced in !64284 (diffs).

I've also added a .to_s call to the environment variable in case it's nil (e.g. in master pipelines).

MR acceptance checklist

This checklist encourages us to confirm any changes have been analyzed to reduce risks in quality, performance, reliability, security, and maintainability.

Related to #345153 (closed).

Edited by Rémy Coutable

Merge request reports