Skip to content

CHORE: Cleanup executors and simulation service#109

Merged
mberz merged 4 commits into
devfrom
chore/cleanup_executors_sim_service
Jun 16, 2026
Merged

CHORE: Cleanup executors and simulation service#109
mberz merged 4 commits into
devfrom
chore/cleanup_executors_sim_service

Conversation

@mberz

@mberz mberz commented Jun 16, 2026

Copy link
Copy Markdown
Contributor

Proposed changes

  • Remove trailing whitespaces
  • Remove unused imports
  • Fix indentation
  • Use f-strings only where relevant

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

This PR focuses on cleanup and formatting improvements across the simulation service and executor implementations to reduce unused dependencies and improve code readability/consistency.

Changes:

  • Reorganized imports and removed several unused imports across simulation and executor modules.
  • Applied whitespace/indentation cleanups and minor formatting consistency fixes.
  • Consolidated discovery service imports and standardized some string formatting.

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 1 comment.

File Description
app/services/simulation_service.py Cleans up imports and formatting; refactors discovery imports and various whitespace/indent fixes.
app/services/executors/local_executor.py Removes unused imports and trailing whitespace/indentation cleanup.
app/services/executors/cloud_executor.py Import reordering/cleanup plus whitespace and minor formatting adjustments.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread app/services/simulation_service.py Outdated

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 3 out of 3 changed files in this pull request and generated 1 comment.

Comment thread app/services/executors/cloud_executor.py
@mberz mberz merged commit 03f3af2 into dev Jun 16, 2026
6 checks passed
@github-project-automation github-project-automation Bot moved this from Backlog to Done in CHORAS planning Jun 16, 2026
@mberz mberz deleted the chore/cleanup_executors_sim_service branch June 16, 2026 13:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants