Speed up builds #393
Open
Speed up builds #393
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
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
1 participant
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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
devandreleaseactions 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.