fedora-java / javapackages

Macros and scripts for Java packaging support
Other
7 stars 15 forks source link

install_pom.py: interpolate properties in current artifactId, groupId and version + add some tests #98

Closed fridrich closed 3 weeks ago

fridrich commented 1 year ago

Some artifacts have things like <artifactId>whatever_${scala.binary.version}</artifactId>. The variable normally is defined either in the current pom or in one of its parents. When generating simplified (dependencies-only) pom, interpolate this kind of variables in the current artifact coordinates too. This would need to be done eventually for mvn_artifact.py too. I tried but did not try a simple elegant way to do it, since the gathering of dependencies happen when the metadata artifact is already intitialized. But eventually one could create a maven artifact from the uart string, interpolate and push the 3 fields back to the metadata artifact. Or create a setter for it in the class.

codecov-commenter commented 1 year ago

Codecov Report

Merging #98 (c3818d3) into master (cc0a7e0) will increase coverage by 0.03%. The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master      #98      +/-   ##
==========================================
+ Coverage   90.96%   91.00%   +0.03%     
==========================================
  Files          45       45              
  Lines        3397     3400       +3     
==========================================
+ Hits         3090     3094       +4     
+ Misses        307      306       -1     
Files Coverage Δ
java-utils/install_pom.py 95.12% <100.00%> (+0.09%) :arrow_up:

... and 1 file with indirect coverage changes

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

fridrich commented 1 year ago

Not sure where the codecov misses come from since I added two tests. Especially them being in files I did not touch. But...

fridrich commented 1 year ago

Not sure where the codecov misses come from since I added two tests. Especially them being in files I did not touch. But...

OK, now I understand, it is from my reproducible build changes from some time ago, not from this PR.

fridrich commented 6 months ago

Ping?

fridrich commented 3 months ago

ping

fridrich commented 3 weeks ago

Ping on this one

mizdebsk commented 3 weeks ago

Thanks for the ping, I will review it soon

mizdebsk commented 3 weeks ago

Looks good.