[
https://issues.apache.org/jira/browse/SLING-9866?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17224568#comment-17224568
]
Stefan Seifert commented on SLING-9866:
---------------------------------------
bq. We can throw an exception in case it is still used so it won't be
unnoticed. Do you know of any projects using this feature?
yes... all our projects which are using sling-initial-content in the bundles,
it's currently activated by default in wcm.io
[aem-global-parent|https://github.com/wcm-io/wcm-io-tooling/tree/develop/maven/aem-global-parent].
in the old times where sling had its on commons json implementation with some
special tweaks it was good to have a validator which uses exactly the same code
base.
for wcm.io aem-global-parent it is easy to switch to a another maven plugin
when it has the same feature set as it is defined centrally - but i cannot
speak for other projects. the mojo is only active when it's called explicitly
in the POM, so it should not be an efficiency problem with projects not using
it.
> Remove Json validator
> ---------------------
>
> Key: SLING-9866
> URL: https://issues.apache.org/jira/browse/SLING-9866
> Project: Sling
> Issue Type: Improvement
> Affects Versions: Sling Maven Plugin 2.4.2
> Reporter: Konrad Windszus
> Priority: Major
> Fix For: Sling Maven Plugin 2.4.4
>
>
> In the context of SLING-653 a generic JSON file validator was added. This is
> not Sling specific and there are specialized Maven plugins like
> https://github.com/sylvainlaurent/yaml-json-validator-maven-plugin which do a
> better job. I would recommend getting rid of
> https://github.com/apache/sling-maven-plugin/blob/master/sling-maven-plugin/src/main/java/org/apache/sling/maven/bundlesupport/ValidationMojo.java
> together with
> https://github.com/apache/sling-maven-plugin/blob/master/sling-maven-plugin/src/main/java/org/apache/sling/maven/bundlesupport/JsonSupport.java
> to reduce the footprint of the sling-maven-plugin.
--
This message was sent by Atlassian Jira
(v8.3.4#803005)
