bosschaert commented on a change in pull request #2:
URL: 
https://github.com/apache/sling-org-apache-sling-javax-activation/pull/2#discussion_r460173604



##########
File path: pom.xml
##########
@@ -77,6 +100,7 @@
         <dependency>
             <groupId>org.osgi</groupId>
             <artifactId>org.osgi.core</artifactId>
+            <version>4.2.0</version>

Review comment:
       OSGi R 4.2 is the minimum OSGi API version that this code needs. It does 
not require anything newer than that right now so why increase the APIs that it 
uses higher than what it actually needs? It will work on any OSGi version after 
that.




----------------------------------------------------------------
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]


Reply via email to