Enhance maven deploy to reference serverId and fix authentication/set…

[rnc]

…tings.xml

For JBS-50

[codecov]

Codecov Report

Attention: Patch coverage is 68.42105% with 6 lines in your changes missing coverage. Please review.

Project coverage is 44.04%. Comparing base (f919901) to head (6e00224).
Report is 16 commits behind head on main.

Files with missing lines	Patch %	Lines
...eploy/mavenrepository/MavenRepositoryDeployer.java	64.70%	3 Missing and 3 partials ⚠️

Additional details and impacted files

@@             Coverage Diff              @@
##               main    #2229      +/-   ##
============================================
+ Coverage     44.02%   44.04%   +0.01%     
  Complexity      811      811              
============================================
  Files           276      276              
  Lines         13464    13474      +10     
  Branches       1360     1363       +3     
============================================
+ Hits           5927     5934       +7     
- Misses         6914     6915       +1     
- Partials        623      625       +2     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[rnc]

This currently isn't passing custom settings.xml locations through ; I think this would need #2224 and then something like 5dc0a2d

[ruhan1]

Looks good to me. Have you tried '-Dmaven.settings=path-to-custom-settings.xml'? for passing custom settings.xml location.

[rnc]

@ruhan1 Thats a good point ; I haven't; that would probably work as well as long as its passed via the task yaml params. I guess I was going for the argument value in 5dc0a2d for consistency

[ruhan1]

Yeah, that looks good. pls merge them so I can try it.