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

Dependency pruning #250

Merged
merged 3 commits into from
Sep 27, 2024
Merged

Dependency pruning #250

merged 3 commits into from
Sep 27, 2024

Conversation

ayushjain17
Copy link
Collaborator

Problem

  1. make run compiles code twice
  2. Redundant dependencies across all crates
  3. Single use dependency specified at workspace Cargo.toml
  4. superposition_types uses diesel as dependency, which causes dependency issues in 3rd party repos using this crate
  5. dependencies in random order
  6. cac_client returning data of types superposition_types::result::Result<T> in cases where the return should neither be a Result type or can simply be a Result<T, String>

Solution

  1. Remove redundant compile step
  2. Remove unused dependencies, mandate #![deny(unused_crate_dependencies)] for all crates
  3. Localise single use dependency to the crates instead of workspace Cargo.toml
  4. make result type as a feature for superposition_types (make diesel optional)
  5. sorted dependencies in alphabetical order
  6. Update return type of cac_client functions to direct T or Result<T, String>

@ayushjain17 ayushjain17 requested a review from a team as a code owner September 27, 2024 10:50
@ayushjain17 ayushjain17 merged commit d01bea6 into main Sep 27, 2024
4 checks passed
@ayushjain17 ayushjain17 deleted the dependencyPruning branch September 27, 2024 11:01
Datron pushed a commit that referenced this pull request Sep 27, 2024
* fix: Cleanup dependency requirements across crates

* fix: Cleanup dependency requirements across crates for examples

* fix: Localise non-repeating crates, instead of workspace
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.

3 participants