Skip to content

chore: remove the logic of Actor._get_default_exit_process for pytest from production code - #654

Merged
vdusek merged 3 commits into
apify:masterfrom
Mantisus:pytest-exit-process
Oct 31, 2025
Merged

vdusek merged 3 commits into
apify:masterfrom
Mantisus:pytest-exit-process

Conversation

@Mantisus

Copy link
Copy Markdown
Collaborator

Issues

Closes: #641

@Mantisus
Mantisus requested review from Pijukatel and vdusek October 30, 2025 08:55
@Mantisus Mantisus self-assigned this Oct 30, 2025
Comment thread tests/unit/conftest.py Outdated
@Mantisus
Mantisus marked this pull request as draft October 30, 2025 13:20
@Mantisus Mantisus changed the title chore: Move the logic of Actor._get_default_exit_process for pytest to the test environment chore: remove the logic of Actor._get_default_exit_process for pytest from production code Oct 30, 2025
@Mantisus
Mantisus requested a review from Pijukatel October 30, 2025 14:15
@Mantisus
Mantisus marked this pull request as ready for review October 30, 2025 14:15

@vdusek vdusek left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

@vdusek
vdusek merged commit 5ece3e7 into apify:master Oct 31, 2025
38 checks passed
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.

Improve Pytest environment handling in Actor._get_default_exit_process

4 participants