Ability to override List/Set/Map property #3

Open
szpak opened this Issue Oct 6, 2014 · 4 comments

Comments

Projects
None yet
3 participants
@szpak
Contributor

szpak commented Oct 6, 2014

It would be nice to be able to override List/Set/Map properties.

@bmuschko

This comment has been minimized.

Show comment
Hide comment
@bmuschko

bmuschko Oct 9, 2014

Contributor

@szpak Would you mind providing test cases for this as well?

Contributor

bmuschko commented Oct 9, 2014

@szpak Would you mind providing test cases for this as well?

@szpak

This comment has been minimized.

Show comment
Hide comment
@szpak

szpak Oct 17, 2014

Contributor

What encoding format did you think about? Groovy like?

-Doverride.extension.myList="['foo','bar']"
-Doverride.extension.myMap="[foo:'bar']"
Contributor

szpak commented Oct 17, 2014

What encoding format did you think about? Groovy like?

-Doverride.extension.myList="['foo','bar']"
-Doverride.extension.myMap="[foo:'bar']"
@bmuschko

This comment has been minimized.

Show comment
Hide comment
@bmuschko

bmuschko Oct 20, 2014

Contributor

I'd say yes. The square brackets should probably be optional and the single quotes omitted. The tricky part is to convert the values to the correct type - especially if the variable definition is declared with def.

Contributor

bmuschko commented Oct 20, 2014

I'd say yes. The square brackets should probably be optional and the single quotes omitted. The tricky part is to convert the values to the correct type - especially if the variable definition is declared with def.

@jmcgarr

This comment has been minimized.

Show comment
Hide comment
@jmcgarr

jmcgarr Apr 18, 2017

We are doing some housecleaning and we consider this plugin "feature complete". Since this is an enhancement, we will accept PR's but won't be able to prioritize this enhancement.

jmcgarr commented Apr 18, 2017

We are doing some housecleaning and we consider this plugin "feature complete". Since this is an enhancement, we will accept PR's but won't be able to prioritize this enhancement.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment