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

build(deps): compile insta in release mode #4671

Merged
merged 1 commit into from
Aug 6, 2024
Merged

Conversation

DonIsaac
Copy link
Collaborator

@DonIsaac DonIsaac commented Aug 5, 2024

Compile insta and its diffing library in release mode. After the initial build, I'm seeing ~1s shaved off cargo test locally.

Compile insta and its diffing library in release mode. After the initial build, I'm seeing ~1s shaved off `cargo test` locally.
@DonIsaac DonIsaac added A-linter Area - Linter C-cleanup Category - technical debt or refactoring. Solution not expected to change behavior C-performance Category - Solution not expected to change functional behavior, only performance and removed C-cleanup Category - technical debt or refactoring. Solution not expected to change behavior labels Aug 5, 2024
@Boshen
Copy link
Member

Boshen commented Aug 6, 2024

NICE! I can't believe that we have so much tests that this is taking into effect.

@Boshen Boshen merged commit f986ec3 into main Aug 6, 2024
14 checks passed
@Boshen Boshen deleted the don/build/insta-dev-release branch August 6, 2024 00:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-linter Area - Linter C-performance Category - Solution not expected to change functional behavior, only performance
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants