`unless separator.empty?` is more readable than `if !separator.empty?`