-
Notifications
You must be signed in to change notification settings - Fork 605
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
Enable profile "cloud" by default #3208
Enable profile "cloud" by default #3208
Conversation
Document disabling cloud profile Fix GH actions build This closes #3201
@royteeuwen Can you have a look at the PR? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
For some weird reason the build order is wrong:
That is despite the fact hat ACS AEM Commons Package depends on the Core Cloud Fragment (in the profile |
I created https://issues.apache.org/jira/browse/MNG-7926 for the ordering issue. |
built "cloud" classifier app package only in profile "cloud"
Actually it was just a misunderstanding from my side, the |
15d5cbc
to
ac25ec2
Compare
@kwin should this PR's build work? Or will this only work once we merge to main? |
Only once we merge, GHA updates are never active from branches/forks due to security concerns. |
Document disabling cloud profile
Fix GH actions build
This closes #3201