[
https://issues.apache.org/jira/browse/SLING-8003?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16682381#comment-16682381
]
ASF GitHub Bot commented on SLING-8003:
---------------------------------------
cziegeler commented on issue #18: SLING-8003 - Interpolate Maven variables
using the Maven Interpolator APIs rather that iterate string replacing
operations
URL:
https://github.com/apache/sling-slingfeature-maven-plugin/pull/18#issuecomment-437581385
Thanks for the patch @simonetripodi
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
> Interpolate Maven variables using the Maven Interpolator APIs rather that
> iterate string replacing operations
> -------------------------------------------------------------------------------------------------------------
>
> Key: SLING-8003
> URL: https://issues.apache.org/jira/browse/SLING-8003
> Project: Sling
> Issue Type: Improvement
> Components: Feature Model, Maven Plugins and Archetypes
> Reporter: Simone Tripodi
> Assignee: David Bosschaert
> Priority: Major
> Fix For: slingfeature-maven-plugin 1.0.0
>
>
> Currently, the {{Substitution}} class iterates over all variables and replace
> them one by one in the whole input string.
> We can improve performances - and avoid writing custom variables interpolator
> - by replacing its use with the {{MavenReaderFilter}} API.
> It will helpful also to avoid reading the Feature file and storing it in a
> String, but keep working with {{Reader}} APIs.
> PR is coming
--
This message was sent by Atlassian JIRA
(v7.6.3#76005)
