Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Remove dependency to commons.lang for Cloud Service #3422

Open
wants to merge 6 commits into
base: master
Choose a base branch
from

Conversation

cziegeler
Copy link
Contributor

This fixes #3421

@cziegeler
Copy link
Contributor Author

@kwin Do you think this is something for 7.0 due to requiring the latest CS release?

@cziegeler
Copy link
Contributor Author

@kwin / @davidjgonzalez Can you please make sure that this gets included in 7.0 ?
Is there anything needed to be changed in this PR?

@kwin
Copy link
Contributor

kwin commented Sep 8, 2024

Fine for me, we don't yet have a branch/timeline for it, though. For now there is just this PR and #3406 which require raising prerequisites.

@kwin kwin added this to the 7.0.0 milestone Sep 8, 2024
@davidjgonzalez
Copy link
Contributor

@cziegeler - got back and looking at this now. Is it correct to stay:

  1. This change is compatible with the current/latest version of AEM as a Cloud Service (which is 2024.9 at the time of writing).
  2. Is this change compatible does not affect AEM 6.5 compatibility.

@cziegeler
Copy link
Contributor Author

@davidjgonzalez Both is correct, requires one of the latest AEM CS versions - the changes are in for a month or so. It doesn't require a new AEM 6.5 version

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Remove dependency on Apache Commons Lang 2 for Cloud Service
3 participants