Skip to content

Speed up builds #393

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

Open
wants to merge 3 commits into
base: dev
Choose a base branch
from
Open

Conversation

sergiolaverde0
Copy link
Contributor

@sergiolaverde0 sergiolaverde0 commented Jan 9, 2025

Hi simjanos, long time no see!

This PR is made to address the slow build times for ARM64 and to fix the various warning in the CI.

On my testing, the first build was done in the amount of time you are used to, 20 minutes for PHP ARM64 image, but the second build used the cached layers and was done in just 2 minutes 40 seconds.

I have not reinstated the Python ARM64, that's up to you for when you decide to enable back. Feel free to test it, and if it is good then we might want to reproduce these changes for the dev and release actions too.

I have also noticed you have been using pre-releases to signal that a beta was out, please let me know if you want me to change this action from manual to be triggered on a pre-release instead.

Cheers.

The older versions of the actions have been deprecated since they
internally use an old version of Node.

Pinning the Ubuntu version prevents it from silently upgrading from
22.04 to 24.04, thus upgrading Python and breaking extra packages. It's
possible to migrate at a more convenient time.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant