Skip to content

Conversation

@leavesster
Copy link
Contributor

  • chore: suppress import warning
  • fix: exit with origin api to exit with real code

@leavesster leavesster requested a review from Copilot March 27, 2025 04:05
Copy link
Contributor

Copilot AI left a comment

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 fixes the issue with exit codes by using the original exit API and suppresses an import warning in the data module.

  • Replace sys.exit(2) with hook.original_exit(2) in executor.py
  • Add a "# noqa: F401" comment to suppress the unused import warning in data.py

Reviewed Changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
executor/python_executor/executor.py Use original exit function to preserve exit code
executor/python_executor/data.py Suppress unused import warning with noqa comment

@leavesster leavesster enabled auto-merge (squash) March 27, 2025 04:07
@leavesster leavesster merged commit 171c02f into main Mar 27, 2025
5 checks passed
@leavesster leavesster deleted the improve branch March 27, 2025 04:08
@oomol-bot oomol-bot mentioned this pull request Mar 27, 2025
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.

2 participants