bosschaert commented on a change in pull request #2:
URL:
https://github.com/apache/sling-org-apache-sling-javax-activation/pull/2#discussion_r460095344
##########
File path: pom.xml
##########
@@ -50,14 +50,37 @@
<plugin>
<groupId>org.apache.felix</groupId>
<artifactId>maven-bundle-plugin</artifactId>
+ <version>5.1.1</version>
Review comment:
There are 2 parent poms: sling 39 and sling-bundle-parent 39. This
project has used sling and I just updated that to the latest of that one. sling
39 does not require migrating to bnd, it's the other one that does.
If you veto this PR because you think it must be migrated to
sling-bundle-parent please let me know, otherwise I suggest that we merge this
PR and the migration to the bnd-maven-plugin and sling-bundle-parent can be
done separately.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
[email protected]
