Skip to content

Conversation

@Alenar
Copy link
Collaborator

@Alenar Alenar commented Oct 11, 2024

Content

This PR simplify the aggregator http server dependencies by providing the dependency container by reference instead of by move, allowing to remove the numerous .clone() used before.

Pre-submit checklist

  • Branch
    • Crates versions are updated (if relevant)
    • Commit sequence broadly makes sense
    • Key commits have useful messages
  • PR
    • No clippy warnings in the CI
    • Self-reviewed the diff
    • Useful pull request description
    • Reviewer requested

@Alenar Alenar self-assigned this Oct 11, 2024
@github-actions
Copy link

github-actions bot commented Oct 11, 2024

Test Results

    4 files  ±0     54 suites  ±0   9m 42s ⏱️ +20s
1 351 tests ±0  1 351 ✅ ±0  0 💤 ±0  0 ❌ ±0 
1 559 runs  ±0  1 559 ✅ ±0  0 💤 ±0  0 ❌ ±0 

Results for commit 4136749. ± Comparison against base commit b541890.

♻️ This comment has been updated with latest results.

@Alenar Alenar force-pushed the djo/aggregator-simplify-http-server-dependencies branch from fc27351 to e151a55 Compare October 11, 2024 11:01
By providing the dependency container by reference instead of by move,
allowing to remove the numerous `.clone()` used before.
@Alenar Alenar force-pushed the djo/aggregator-simplify-http-server-dependencies branch from e151a55 to c42cbb1 Compare October 11, 2024 13:01
* mithril-aggregator from `0.5.79` to `0.5.80`
Copy link
Member

@jpraynaud jpraynaud left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM 👍

@Alenar Alenar temporarily deployed to testing-preview October 11, 2024 13:35 — with GitHub Actions Inactive
@Alenar Alenar temporarily deployed to testing-sanchonet October 11, 2024 13:35 — with GitHub Actions Inactive
@Alenar Alenar merged commit 0bc09f4 into main Oct 11, 2024
48 checks passed
@Alenar Alenar deleted the djo/aggregator-simplify-http-server-dependencies branch October 11, 2024 13:42
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.

4 participants