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 package and upload artifacts steps on CircleCI #1269

Merged
merged 1 commit into from
Mar 7, 2025

Conversation

DanRStevens
Copy link
Collaborator

We don't really use these. They would make more sense with automated release packaging, and only if we automated release packages for those platforms. They might make sense if people were following along with the builds and downloading binary packages, rather than building for themselves. So far though, there is no indication anyone is doing this, or is even aware that it was possible.

On the other hand, the CircleCI usage statistics show we are way over for both compute and storage. Disabling build artifacts should drastically cut down on storage. It may also cut down quite a bit on CPU usage too, since upload can sometimes be a bit slow. A recent MacOS build showed 43s upload on a 2:13 build. It's made worse by MacOS being the platform that costs the most credits. So far we aren't paying for them, but we should probably still try to stay within the limits of the free usage tier.

We don't really use these. They would make more sense with automated release packaging, and only if we automated release packages for those platforms. They might make sense if people were following along with the builds and downloading binary packages, rather than building for themselves. So far though, there is no indication anyone is doing this, or is even aware that it was possible.

On the other hand, the CircleCI usage statistics show we are way over for both compute and storage. Disabling build artifacts should drastically cut down on storage. It may also cut down quite a bit on CPU usage too, since upload can sometimes be a bit slow. A recent MacOS build showed 43s upload on a 2:13 build. It's made worse by MacOS being the platform that costs the most credits. So far we aren't paying for them, but we should probably still try to stay within the limits of the free usage tier.
@DanRStevens DanRStevens merged commit 660a5a4 into main Mar 7, 2025
8 checks passed
@DanRStevens DanRStevens deleted the circleCiNoUploadArtifacts branch March 7, 2025 03:19
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