[GitHub] [sling-org-apache-sling-feature-cpconverter] kwin edited a comment on pull request #70: SLING-10127 emit all converted packages to the local Maven repository

2021-04-13 Thread GitBox
kwin edited a comment on pull request #70: URL: https://github.com/apache/sling-org-apache-sling-feature-cpconverter/pull/70#issuecomment-818556608 @karlpauls `dropContent` is only an internal field name and can imho be renamed to reflect that this is only about not referencing it in the

[GitHub] [sling-org-apache-sling-feature-cpconverter] kwin edited a comment on pull request #70: SLING-10127 emit all converted packages to the local Maven repository

2021-04-13 Thread GitBox
kwin edited a comment on pull request #70: URL: https://github.com/apache/sling-org-apache-sling-feature-cpconverter/pull/70#issuecomment-818556608 @karlpauls `dropContent` is only an internal flag and can imho be renamed to reflect that this is only about not referencing it in the

[GitHub] [sling-org-apache-sling-feature-cpconverter] kwin commented on pull request #70: SLING-10127 emit all converted packages to the local Maven repository

2021-04-13 Thread GitBox
kwin commented on pull request #70: URL: https://github.com/apache/sling-org-apache-sling-feature-cpconverter/pull/70#issuecomment-818556608 @karlpauls `dropContent` is only an internal flag and can imho be renamed to reflect that this is only about not referencing it in the feature

[GitHub] [sling-org-apache-sling-jcr-contentloader] sonarcloud[bot] removed a comment on pull request #6: SLING-10297 expose PathEntry as API

2021-04-13 Thread GitBox
sonarcloud[bot] removed a comment on pull request #6: URL: https://github.com/apache/sling-org-apache-sling-jcr-contentloader/pull/6#issuecomment-818552028 Kudos, SonarCloud Quality Gate passed!

[GitHub] [sling-org-apache-sling-jcr-contentloader] sonarcloud[bot] commented on pull request #6: SLING-10297 expose PathEntry as API

2021-04-13 Thread GitBox
sonarcloud[bot] commented on pull request #6: URL: https://github.com/apache/sling-org-apache-sling-jcr-contentloader/pull/6#issuecomment-818552714 Kudos, SonarCloud Quality Gate passed!

[GitHub] [sling-org-apache-sling-jcr-contentloader] sonarcloud[bot] commented on pull request #6: SLING-10297 expose PathEntry as API

2021-04-13 Thread GitBox
sonarcloud[bot] commented on pull request #6: URL: https://github.com/apache/sling-org-apache-sling-jcr-contentloader/pull/6#issuecomment-818552028 Kudos, SonarCloud Quality Gate passed!

[GitHub] [sling-org-apache-sling-jcr-contentloader] sonarcloud[bot] removed a comment on pull request #6: SLING-10297 expose PathEntry as API

2021-04-13 Thread GitBox
sonarcloud[bot] removed a comment on pull request #6: URL: https://github.com/apache/sling-org-apache-sling-jcr-contentloader/pull/6#issuecomment-818549249 Kudos, SonarCloud Quality Gate passed!

[GitHub] [sling-org-apache-sling-jcr-contentloader] sonarcloud[bot] commented on pull request #6: SLING-10297 expose PathEntry as API

2021-04-13 Thread GitBox
sonarcloud[bot] commented on pull request #6: URL: https://github.com/apache/sling-org-apache-sling-jcr-contentloader/pull/6#issuecomment-818549249 Kudos, SonarCloud Quality Gate passed!

[GitHub] [sling-org-apache-sling-jcr-contentloader] kwin opened a new pull request #6: SLING-10297 expose PathEntry as API

2021-04-13 Thread GitBox
kwin opened a new pull request #6: URL: https://github.com/apache/sling-org-apache-sling-jcr-contentloader/pull/6 -- 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

[jira] [Commented] (SLING-10289) Sling Dynamic Include 3.2.0 configuration of arrays broken

2021-04-13 Thread Robert Munteanu (Jira)
[ https://issues.apache.org/jira/browse/SLING-10289?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=1732#comment-1732 ] Robert Munteanu commented on SLING-10289: - Good catch [~AndreasWurm]. Would you have the time to

[jira] [Updated] (SLING-10289) Sling Dynamic Include 3.2.0 configuration of arrays broken

2021-04-13 Thread Robert Munteanu (Jira)
[ https://issues.apache.org/jira/browse/SLING-10289?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Robert Munteanu updated SLING-10289: Fix Version/s: Dynamic Include 3.2.2 > Sling Dynamic Include 3.2.0 configuration of

[jira] [Resolved] (SLING-10278) Treat install and config folders the same

2021-04-13 Thread Konrad Windszus (Jira)
[ https://issues.apache.org/jira/browse/SLING-10278?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Konrad Windszus resolved SLING-10278. - Resolution: Fixed Fixed in

[GitHub] [sling-org-apache-sling-feature-cpconverter] kwin merged pull request #72: SLING-10278 allow configurations below "install" and bundles below

2021-04-13 Thread GitBox
kwin merged pull request #72: URL: https://github.com/apache/sling-org-apache-sling-feature-cpconverter/pull/72 -- 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

[GitHub] [sling-org-apache-sling-jcr-repoinit] sonarcloud[bot] removed a comment on pull request #14: SLING-10281 revert SLING-9449

2021-04-13 Thread GitBox
sonarcloud[bot] removed a comment on pull request #14: URL: https://github.com/apache/sling-org-apache-sling-jcr-repoinit/pull/14#issuecomment-816482290 Kudos, SonarCloud Quality Gate passed!

[GitHub] [sling-org-apache-sling-jcr-repoinit] sonarcloud[bot] commented on pull request #14: SLING-10281 revert SLING-9449

2021-04-13 Thread GitBox
sonarcloud[bot] commented on pull request #14: URL: https://github.com/apache/sling-org-apache-sling-jcr-repoinit/pull/14#issuecomment-818518285 Kudos, SonarCloud Quality Gate passed! [](https://sonarcloud.io/project/issues?id=apache_sling-org-apache-sling-jcr-repoinit=14=false=BUG)

[GitHub] [sling-org-apache-sling-feature-cpconverter] karlpauls commented on pull request #70: SLING-10127 emit all converted packages to the local Maven repository

2021-04-13 Thread GitBox
karlpauls commented on pull request #70: URL: https://github.com/apache/sling-org-apache-sling-feature-cpconverter/pull/70#issuecomment-818516741 @kwin, I don't disagree that it might be a mistake to ignore "content" content packages - however, that is what this option is specifically for

[GitHub] [sling-org-apache-sling-feature-cpconverter] sonarcloud[bot] commented on pull request #70: SLING-10127 emit all converted packages to the local Maven repository

2021-04-13 Thread GitBox
sonarcloud[bot] commented on pull request #70: URL: https://github.com/apache/sling-org-apache-sling-feature-cpconverter/pull/70#issuecomment-818516185 SonarCloud Quality Gate failed.

[GitHub] [sling-org-apache-sling-feature-cpconverter] sonarcloud[bot] removed a comment on pull request #70: SLING-10127 emit all converted packages to the local Maven repository

2021-04-13 Thread GitBox
sonarcloud[bot] removed a comment on pull request #70: URL: https://github.com/apache/sling-org-apache-sling-feature-cpconverter/pull/70#issuecomment-817685935 SonarCloud Quality Gate failed.

[GitHub] [sling-org-apache-sling-feature-cpconverter] sonarcloud[bot] removed a comment on pull request #72: SLING-10278 allow configurations below "install" and bundles below

2021-04-13 Thread GitBox
sonarcloud[bot] removed a comment on pull request #72: URL: https://github.com/apache/sling-org-apache-sling-feature-cpconverter/pull/72#issuecomment-818502836 SonarCloud Quality Gate failed.

[GitHub] [sling-org-apache-sling-feature-cpconverter] sonarcloud[bot] commented on pull request #72: SLING-10278 allow configurations below "install" and bundles below

2021-04-13 Thread GitBox
sonarcloud[bot] commented on pull request #72: URL: https://github.com/apache/sling-org-apache-sling-feature-cpconverter/pull/72#issuecomment-818516063 SonarCloud Quality Gate failed.

[jira] [Updated] (SLING-10297) Expose org.apache.sling.jcr.contentloader.internal.PathEntry as API

2021-04-13 Thread Konrad Windszus (Jira)
[ https://issues.apache.org/jira/browse/SLING-10297?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Konrad Windszus updated SLING-10297: Description: To ease parsing the {{Sling-Initial-Content}} header outside the OSGi

[jira] [Created] (SLING-10297) Expose org.apache.sling.jcr.contentloader.internal.PathEntry as API

2021-04-13 Thread Konrad Windszus (Jira)
Konrad Windszus created SLING-10297: --- Summary: Expose org.apache.sling.jcr.contentloader.internal.PathEntry as API Key: SLING-10297 URL: https://issues.apache.org/jira/browse/SLING-10297 Project:

[GitHub] [sling-org-apache-sling-feature-cpconverter] sonarcloud[bot] removed a comment on pull request #72: SLING-10278 allow configurations below "install" and bundles below

2021-04-13 Thread GitBox
sonarcloud[bot] removed a comment on pull request #72: URL: https://github.com/apache/sling-org-apache-sling-feature-cpconverter/pull/72#issuecomment-817970182 SonarCloud Quality Gate failed.

[GitHub] [sling-org-apache-sling-feature-cpconverter] sonarcloud[bot] commented on pull request #72: SLING-10278 allow configurations below "install" and bundles below

2021-04-13 Thread GitBox
sonarcloud[bot] commented on pull request #72: URL: https://github.com/apache/sling-org-apache-sling-feature-cpconverter/pull/72#issuecomment-818502836 SonarCloud Quality Gate failed.

[GitHub] [sling-org-apache-sling-feature-cpconverter] kwin commented on pull request #72: SLING-10278 allow configurations below "install" and bundles below

2021-04-13 Thread GitBox
kwin commented on pull request #72: URL: https://github.com/apache/sling-org-apache-sling-feature-cpconverter/pull/72#issuecomment-818499494 @cziegeler Incorporated your feedback in

[jira] [Commented] (SLING-10278) Treat install and config folders the same

2021-04-13 Thread Carsten Ziegeler (Jira)
[ https://issues.apache.org/jira/browse/SLING-10278?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17319949#comment-17319949 ] Carsten Ziegeler commented on SLING-10278: -- Thanks, PR looks basically good to me - can we

Re: Release of Servlets Resolver and Scripting bundles

2021-04-13 Thread Carsten Ziegeler
I tend to agree that there is not that much value in moving this to the api. I don't see any use cases for users of Sling's resource api to also use this class. Apart from the reasons mentioned, ResourceType talks about versioning - a concept we currently don't have in the api. In addition

<    1   2