[
https://issues.apache.org/jira/browse/SLING-7798?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Stefan Seifert updated SLING-7798:
----------------------------------
Description:
follow-up from SLING-7312 and discussion from the [mailing
list|https://lists.apache.org/thread.html/44cac2f3c9454a308b70adb9c73a8e9a928d9c145a12f82f8a3a0ef1@%3Cdev.sling.apache.org%3E].
i will also start a vote on the mailing list about this, but this ticket
describes the steps that are required.
about the jetbrains annotations:
* documentation:
[https://www.jetbrains.com/help/idea/nullable-and-notnull-annotations.html]
* artifact: [http://repo1.maven.org/maven2/org/jetbrains/annotations/16.0.2/]
* although from jetbrans created for IntelliJ, they are also support by most
other tooling (e.g. findbugs, sonar, configurable in Eclipse)
* apache 2.0 licensed: [https://github.com/JetBrains/java-annotations]
this search turned out that these modules are currently using JSR-305
annotations and need to be converted:
||Module||Switch||Released||
|org-apache-sling-api|(/)| (/)|
|org-apache-sling-auth-core|(/) removed unused dependency| -|
|org-apache-sling-caconfig-api|(/)| (x)|
|org-apache-sling-caconfig-impl|(/)| (x)|
|org-apache-sling-caconfig-spi|(/)| (x)|
|org-apache-sling-commons-clam|(/) no release yet| -|
|org-apache-sling-commons-messaging|(/) no release yet| -|
|org-apache-sling-commons-messaging-mail|(/) no release yet| -|
|org-apache-sling-discovery-commons|(/)| (x)|
|org-apache-sling-distribution-api|(/)| (x)|
|org-apache-sling-distribution-avro-serializer|(/) no release yet| -|
|org-apache-sling-distribution-core|(/) unit test fails (already before)| (/)|
|org-apache-sling-distribution-kryo-serializer|(/) no release yet| -|
|org-apache-sling-installer-core|(/)| (/)|
|org-apache-sling-jcr-api|(/)| (x)|
|org-apache-sling-jcr-resource|(/)| (/)|
|org-apache-sling-jms|(/)| (x)|
|org-apache-sling-jobs|(/)| (x)|
|org-apache-sling-jobs-it-services|(/) no release required| -|
|org-apache-sling-models-api|(/)| (x)|
|org-apache-sling-models-impl|(/)| (x)|
|org-apache-sling-models-jacksonexporter|(/)| (x)|
|org-apache-sling-models-validation-impl|(/) no release yet| -|
|org-apache-sling-mom|(/)| (x)|
|org-apache-sling-oak-restrictions|(/)| (x)|
|org-apache-sling-resource-presence|(/)| (x)|
|org-apache-sling-resourceresolver|(/)| (/)|
|org-apache-sling-scripting-core|(/)| (x)|
|org-apache-sling-scripting-sightly|(/)| (x)|
|org-apache-sling-scripting-thymeleaf|(/)| (x)|
|org-apache-sling-tracer|(/)| (x)|
|org-apache-sling-validation-api|(/)| (x)|
|org-apache-sling-validation-core|(/)| (x)|
|org-apache-sling-validation-test-services|(/) no release required| -|
|org-apache-sling-xss|(/)| (x)|
|parent|(/)| (/)|
|whiteboard/encrypt|(/) no release required| -|
|whiteboard/project-archetype/src/main/resources/archetype-resources|(/) no
release required| -|
|whiteboard/scripting-resolver/org-apache-sling-scripting-resolver|(/) no
release required| -|
steps to do:
# convert java sources and poms to jetbrains annotations and update package
versions with the help of the {{convert-annotations.sh}} from OAK-7511 provided
by [~reschke] described here
# update sling documentation:
[https://sling.apache.org/documentation/development/jsr-305.html]
was:
follow-up from SLING-7312 and discussion from the [mailing
list|https://lists.apache.org/thread.html/44cac2f3c9454a308b70adb9c73a8e9a928d9c145a12f82f8a3a0ef1@%3Cdev.sling.apache.org%3E].
i will also start a vote on the mailing list about this, but this ticket
describes the steps that are required.
about the jetbrains annotations:
* documentation:
[https://www.jetbrains.com/help/idea/nullable-and-notnull-annotations.html]
* artifact: [http://repo1.maven.org/maven2/org/jetbrains/annotations/16.0.2/]
* although from jetbrans created for IntelliJ, they are also support by most
other tooling (e.g. findbugs, sonar, configurable in Eclipse)
* apache 2.0 licensed: [https://github.com/JetBrains/java-annotations]
this search turned out that these modules are currently using JSR-305
annotations and need to be converted:
|org-apache-sling-api|(/)|
|org-apache-sling-auth-core|(/) removed unused dependency|
|org-apache-sling-caconfig-api|(/)|
|org-apache-sling-caconfig-impl|(/)|
|org-apache-sling-caconfig-spi|(/)|
|org-apache-sling-commons-clam|(/)|
|org-apache-sling-commons-messaging|(/) no release yet|
|org-apache-sling-commons-messaging-mail|(/) no release yet|
|org-apache-sling-discovery-commons|(/)|
|org-apache-sling-distribution-api|(/)|
|org-apache-sling-distribution-avro-serializer|(/) no release yet|
|org-apache-sling-distribution-core|(/) unit test fails (already before)|
|org-apache-sling-distribution-kryo-serializer|(/) no release yet|
|org-apache-sling-installer-core|(/)|
|org-apache-sling-jcr-api|(/)|
|org-apache-sling-jcr-resource|(/)|
|org-apache-sling-jms|(/)|
|org-apache-sling-jobs|(/)|
|org-apache-sling-jobs-it-services|(/) no release required|
|org-apache-sling-models-api|(/)|
|org-apache-sling-models-impl|(/)|
|org-apache-sling-models-jacksonexporter|(/)|
|org-apache-sling-models-validation-impl|(/)|
|org-apache-sling-mom|(/)|
|org-apache-sling-oak-restrictions|(/)|
|org-apache-sling-resource-presence|(/)|
|org-apache-sling-resourceresolver|(/)|
|org-apache-sling-scripting-core|(/)|
|org-apache-sling-scripting-sightly|(/)|
|org-apache-sling-scripting-thymeleaf|(/)|
|org-apache-sling-tracer|(/)|
|org-apache-sling-validation-api|(/)|
|org-apache-sling-validation-core|(/)|
|org-apache-sling-validation-test-services|(/) no release required|
|org-apache-sling-xss|(/)|
|parent|(/)|
|whiteboard/encrypt|(/) no release required|
|whiteboard/project-archetype/src/main/resources/archetype-resources|(/) no
release required|
|whiteboard/scripting-resolver/org-apache-sling-scripting-resolver|(/) no
release required|
steps to do:
# convert java sources and poms to jetbrains annotations and update package
versions with the help of the {{convert-annotations.sh}} from OAK-7511 provided
by [~reschke] described here
# update sling documentation:
[https://sling.apache.org/documentation/development/jsr-305.html]
> Switch from JSR-305 annotations to JetBrains Nullable/NotNull annotations
> --------------------------------------------------------------------------
>
> Key: SLING-7798
> URL: https://issues.apache.org/jira/browse/SLING-7798
> Project: Sling
> Issue Type: Improvement
> Components: General
> Reporter: Stefan Seifert
> Assignee: Stefan Seifert
> Priority: Major
> Fix For: JCR API 2.4.2, Auth Core 1.4.4, API 2.18.4, Sling Models
> API 1.3.8, Content Distribution Core 0.3.0, Content Distribution API 0.4.0,
> Validation API 1.0.2, Sling Models Validation Impl 1.0.0, Log Tracer 1.0.8,
> Commons Messaging 1.0.0, Commons Messaging Mail 1.0.0, Scripting Thymeleaf
> 2.0.0, Oak Restrictions 1.0.2, Context-Aware Configuration API 1.1.2, MoM API
> 1.0.2, MoM JMS 1.0.2, MoM Jobs 1.0.2, Resource Presence 0.0.4, Discovery
> Commons 1.0.24, Context-Aware Configuration SPI 1.3.4, Sling Models Jackson
> Exporter 1.0.10, Validation Core 1.0.6, Parent 34, Scripting Core 2.0.56,
> Installer Core 3.9.0, Sling Models Impl 1.4.10, Context-Aware Configuration
> Impl 1.4.14, Commons Clam 1.0.0, Resource Resolver 1.6.6, JCR Resource
> 3.0.16, XSS Protection API 2.0.14, Scripting HTL Engine 1.0.56-1.4.0
>
>
> follow-up from SLING-7312 and discussion from the [mailing
> list|https://lists.apache.org/thread.html/44cac2f3c9454a308b70adb9c73a8e9a928d9c145a12f82f8a3a0ef1@%3Cdev.sling.apache.org%3E].
> i will also start a vote on the mailing list about this, but this ticket
> describes the steps that are required.
> about the jetbrains annotations:
> * documentation:
> [https://www.jetbrains.com/help/idea/nullable-and-notnull-annotations.html]
> * artifact: [http://repo1.maven.org/maven2/org/jetbrains/annotations/16.0.2/]
> * although from jetbrans created for IntelliJ, they are also support by most
> other tooling (e.g. findbugs, sonar, configurable in Eclipse)
> * apache 2.0 licensed: [https://github.com/JetBrains/java-annotations]
> this search turned out that these modules are currently using JSR-305
> annotations and need to be converted:
> ||Module||Switch||Released||
> |org-apache-sling-api|(/)| (/)|
> |org-apache-sling-auth-core|(/) removed unused dependency| -|
> |org-apache-sling-caconfig-api|(/)| (x)|
> |org-apache-sling-caconfig-impl|(/)| (x)|
> |org-apache-sling-caconfig-spi|(/)| (x)|
> |org-apache-sling-commons-clam|(/) no release yet| -|
> |org-apache-sling-commons-messaging|(/) no release yet| -|
> |org-apache-sling-commons-messaging-mail|(/) no release yet| -|
> |org-apache-sling-discovery-commons|(/)| (x)|
> |org-apache-sling-distribution-api|(/)| (x)|
> |org-apache-sling-distribution-avro-serializer|(/) no release yet| -|
> |org-apache-sling-distribution-core|(/) unit test fails (already before)| (/)|
> |org-apache-sling-distribution-kryo-serializer|(/) no release yet| -|
> |org-apache-sling-installer-core|(/)| (/)|
> |org-apache-sling-jcr-api|(/)| (x)|
> |org-apache-sling-jcr-resource|(/)| (/)|
> |org-apache-sling-jms|(/)| (x)|
> |org-apache-sling-jobs|(/)| (x)|
> |org-apache-sling-jobs-it-services|(/) no release required| -|
> |org-apache-sling-models-api|(/)| (x)|
> |org-apache-sling-models-impl|(/)| (x)|
> |org-apache-sling-models-jacksonexporter|(/)| (x)|
> |org-apache-sling-models-validation-impl|(/) no release yet| -|
> |org-apache-sling-mom|(/)| (x)|
> |org-apache-sling-oak-restrictions|(/)| (x)|
> |org-apache-sling-resource-presence|(/)| (x)|
> |org-apache-sling-resourceresolver|(/)| (/)|
> |org-apache-sling-scripting-core|(/)| (x)|
> |org-apache-sling-scripting-sightly|(/)| (x)|
> |org-apache-sling-scripting-thymeleaf|(/)| (x)|
> |org-apache-sling-tracer|(/)| (x)|
> |org-apache-sling-validation-api|(/)| (x)|
> |org-apache-sling-validation-core|(/)| (x)|
> |org-apache-sling-validation-test-services|(/) no release required| -|
> |org-apache-sling-xss|(/)| (x)|
> |parent|(/)| (/)|
> |whiteboard/encrypt|(/) no release required| -|
> |whiteboard/project-archetype/src/main/resources/archetype-resources|(/) no
> release required| -|
> |whiteboard/scripting-resolver/org-apache-sling-scripting-resolver|(/) no
> release required| -|
> steps to do:
> # convert java sources and poms to jetbrains annotations and update package
> versions with the help of the {{convert-annotations.sh}} from OAK-7511
> provided by [~reschke] described here
> # update sling documentation:
> [https://sling.apache.org/documentation/development/jsr-305.html]
--
This message was sent by Atlassian JIRA
(v7.6.3#76005)
