Skip to content

Remove Enso's simulator support#4294

Merged
jmg-duarte merged 3 commits intomainfrom
jmgd/enso
Mar 30, 2026
Merged

Remove Enso's simulator support#4294
jmg-duarte merged 3 commits intomainfrom
jmgd/enso

Conversation

@jmg-duarte
Copy link
Copy Markdown
Contributor

Description

Remove the Enso Trade Simulator backend. It's unused and adds unnecessary code to maintain.

Changes

  • Remove the unused Enso simulator backend — deleted the enso/ module and cleaned up all references in configs, driver, and simulator crates.

How to test

cargo nextest run -p simulator -p configs -p driver

@jmg-duarte jmg-duarte requested a review from a team as a code owner March 24, 2026 21:45
Copy link
Copy Markdown
Contributor

@gemini-code-assist gemini-code-assist bot left a comment

Choose a reason for hiding this comment

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

Code Review

This pull request completely removes the Enso simulator integration, including its configuration, implementation, and all related code and references across the configs, driver, and simulator crates. No feedback to provide.

Copy link
Copy Markdown
Contributor

@m-sz m-sz left a comment

Choose a reason for hiding this comment

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

LGTM.

If we further decide to remove Tenderly, the simulator struct would have to be revisited.

@jmg-duarte jmg-duarte added this pull request to the merge queue Mar 30, 2026
Merged via the queue into main with commit 3dd0844 Mar 30, 2026
19 checks passed
@jmg-duarte jmg-duarte deleted the jmgd/enso branch March 30, 2026 17:29
@github-actions github-actions bot locked and limited conversation to collaborators Mar 30, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants