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

Reorganisation #175

Merged
merged 17 commits into from
Mar 8, 2021
Merged

Reorganisation #175

merged 17 commits into from
Mar 8, 2021

Conversation

piakushin
Copy link
Member

@piakushin piakushin commented Feb 21, 2021

Changing bob project structure:

  • crate bob
    • bob
    • bob-common
    • bob-backend
    • bob-grpc
    • bob-tools

@piakushin
Copy link
Member Author

Compilation time comparison, 8 cores 4.6GHz:

test/branch current master
cargo build 1m 31s 1m 39s
cargo build --release 2m 18s 2m 23s
cargo check (full) 1m 03s 1m 02s
cargo check(1 change in grinder.rs) 1.39s 2.66s

Cargo check completes faster on workspace because it doesn't recompile all mods, only subcrate with changes.

@piakushin piakushin marked this pull request as ready for review March 5, 2021 11:17
@piakushin piakushin added feature New feature or request high priority labels Mar 5, 2021
@piakushin piakushin merged commit 043dc33 into master Mar 8, 2021
@piakushin piakushin deleted the reorganisation branch March 8, 2021 12:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature New feature or request high priority
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant