Skip to content

Conversation

@konstin
Copy link
Member

@konstin konstin commented Nov 19, 2025

No description provided.

@konstin konstin added the internal A refactor or improvement that is not user-facing label Nov 19, 2025
@konstin konstin temporarily deployed to uv-test-registries November 19, 2025 18:06 — with GitHub Actions Inactive
@konstin konstin temporarily deployed to uv-test-registries November 19, 2025 18:33 — with GitHub Actions Inactive
@konstin
Copy link
Member Author

konstin commented Nov 19, 2025

Windows seems unchanged, Linux goes from 6:xx min to 4:xx min

@zanieb
Copy link
Member

zanieb commented Nov 19, 2025

Can we use this for the binary builds we use for integration tests too?

Is there risk to using these fast builds for tests? Are we going to lose test coverage?

@konstin
Copy link
Member Author

konstin commented Nov 19, 2025

The fast-build profile strip debug info, so we'd lose information if we running a coverage tools like coverage.py, but I'm not aware that we're using such a tool. No debug info also means that if there's a crash, we don't get a backtrace, similar to our release builds. If that's a good tradeoff IMHO depends on how much faster it is.

We don't use any functional coverage in our tests though, the binaries are only a bit more optimized and stripped of debuginfo, basically half way between a regular debug builds and full release builds.

@konstin
Copy link
Member Author

konstin commented Nov 19, 2025

Can we use this for the binary builds we use for integration tests too?

Added, let's see how it goes

CI is too flaky too matter, and we have merge gates
@konstin konstin force-pushed the konsti/cargo-fast-build branch from abac574 to 116a446 Compare November 21, 2025 11:51
@konstin konstin temporarily deployed to uv-test-registries November 21, 2025 11:55 — with GitHub Actions Inactive
@konstin konstin temporarily deployed to uv-test-registries November 21, 2025 12:29 — with GitHub Actions Inactive
@konstin konstin temporarily deployed to uv-test-registries November 21, 2025 14:04 — with GitHub Actions Inactive
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

internal A refactor or improvement that is not user-facing

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants