Automatically include contributors in pom.xml file about shipkit

[mstachniuk]

I think it can be done without any pom.xml template - it will be hard to keep template and pom.xml in sync. I will look more deeply on it in April.

[mockitoguy]

I've created initial pom plugin. Added a TODO to the code with some examples how to approach the problem :)

[mstachniuk]

Design question
I fought about how to solve this problem. GitHubContributorsProvider.mapContributorsToGitHubUser(...) always ask only for contributors between two commits (usually last released version and HEAD). My first idea was to call this method inside PomCustomizer.customizePom(...), but then only last contributors will be added to the pom.xml (and hardcoded in gradle/release.gradle).

In my opinion it's better to update in some previous step (before generatePomFileForJavaLibraryPublication) a property ext.pom_contributors in gradle/release.gradle and add new contributors to the list. Then we need to adjust logic in PomCustomizer.customizePom(...) a little bit (to handle more info about contributors) and continous-delivery-drone will commit this file together with release-notes. In this approach we reduce a number of requests to github.com, speed up a build and are sure that we don't skip anybody in pom.

@szczepiq WDYT?

[mockitoguy]

Let's optimize for:

• least amount of requests to GitHub
• minimum overhead to the build time

We have a couple of options:

• we just maintain the mapping in a file, it would be manually maintained
• we add a discrete step (task) that gets the data before we update pom (this is what you're suggesting, right?)
• do some smart caching if possible, commits pushed to main dev branches don't change :)

I've started to use latest version of mockito release tools in mockito and noticed that the generation of notable release notes adds overhead due to getting the user mappings for each notable version. For the time being I'll simplify it and avoid requesting user mappings for notable release notes. They don't need it because the notable notes don't have authors (for now, we can change it if we see fit).

Thank you for working on this!!!