new nested params `load` feature breaks workflow about nf-test

[lukfor]

Thanks for reporting this issue 👍 I will look into it and remove the variables from the nested maps.

[aaron-fishman-achillestx]

@lukfor perhaps you can do this by putting these vars in the delegate of the closures, rather than in the closure's themselves?

[aaron-fishman-achillestx]

or could be manually removed as well I guess.

[lukfor]

@lukfor perhaps you can do this by putting these vars in the delegate of the closures, rather than in the closure's themselves?

I remember I had some problems with this approach and variable substitution. But I will look into that the next days.

[birnbera]

As a workaround, I found that I can replicate the load functionality without the extra keys side effect using:

params {
    delegate.putAll(path('path/to/params.json').json)
    additional_key = 'some value'
}

I believe this works because the params map is the delegate of the params closure and forwards the putAll method to the underlying map.

[lukfor]

PR #117 will fix also this issue.