artifactory-user-plugins
artifactory-user-plugins copied to clipboard
Add all repos if non given
like cleanup script
Hi @derhecht , thank you for your contribution.
I'm not very comfortable with adding behavior when some configuration value is missing. I think It can hide configuration mistakes and provide an undesired result. This is specially important in cases like this where we are deleting things from the server.
I would like to suggest the usage of a key word, something like ALL
, to be used by the user to set the repo/path list parameter when looking for the behavior added by you. We also need to document that as a possible configuration value in the README file.
Excellent change! Picked up for my RT
Hi @derhecht , thank you for your contribution.
I'm not very comfortable with adding behavior when some configuration value is missing. I think It can hide configuration mistakes and provide an undesired result. This is specially important in cases like this where we are deleting things from the server.
I would like to suggest the usage of a key word, something like
ALL
, to be used by the user to set the repo/path list parameter when looking for the behavior added by you. We also need to document that as a possible configuration value in the README file.
Hi @elioengcomp may have a look at this approach?
ok, added email address to my account, signed CLA, recheck seems not to work...
got it. finally.