Simplify configuration required to upload maven artifacts about drone-artifactory HOT 5 CLOSED

Hi there,

It's quite a good idea. Let's me resume. Would it be enough to:

• If you specify a pom by property, upload it automatically as if it were a file.
• Changing the default files value to be [target/*.jar]

I would like to keep the files property instead of jar if you don't mind cause I use the plugin a lot for war as well and all sorts of -sources- and -doc-

What we could do as well is to accept a single entry for it as well as a list.

What do you think?

from drone-artifactory.

Hi @athieriot, thanks for the comments. I think that's reasonable. I'd like to have a go at this but will wait till your pull request is merged as i think conflicts would be likely due to the amount of refactoring

Thanks!

from drone-artifactory.

Thank you !

from drone-artifactory.

Hi @athieriot, I've raised pull request #7

On reflection I don't think it's worth adding a default value for target/*.jar....it's not functionality i need at the moment either. What do you think? I think it's a bit of a specific scenario and assumes that the user is using maven rather than being build tool agnostic.

from drone-artifactory.

If it's something you don't really need then I think you are right.

Even, I would be quite curious to see if the plugin, in its current state, is working fine with non-Maven artefacts

from drone-artifactory.