GH1106: Multiple values passed to WithProperty are overridden #2110
Conversation
|
|
devlead
force-pushed
the
feature/gh-1106-multiple-values-withproperty
branch
from
770732d
to
95e2547
Compare
augustoproiete
changed the title
augustoproiete
force-pushed
the
feature/gh-1106-multiple-values-withproperty
branch
from
95e2547
to
0a5aa03
Compare
…mma separated list
augustoproiete
force-pushed
the
feature/gh-1106-multiple-values-withproperty
branch
from
0a5aa03
to
6f748a0
Compare
There was a problem hiding this comment.
@Caopi Thanks you for your contribution and sorry for the late answer. We're now at a point where we can change this behavior in an upcoming release.
We would prefer to use semicolon instead of comas for separating the values as mentioned in the comment. Additionally there should be some integration tests for this, which makes sure that this actually works together with MsBuild.
Are you still in a position to continue with this PR or prefer for someone else to take over?
Co-authored-by: Pascal Berger <pascal.berger@gmail.com>
|
Hey Pascal, sorry for the late reply as well. I've quickly accepted your suggestion of course, but I would prefer if someone else could take over the integration test part. Haven't worked with C# and Cake in a while nowadays and don't have a lot of time to commit to this right now, sorry! |
gep13
changed the title
Changed MSBuildExtension's WithProperty to forward multiple parameters as a comma separated list surrounded by quotes. Single parameter behavior is unchanged to before and does not have the added quotes.
This should fix #1106. Check the comments there for more details.