Skip to content

Conversation

@VegetarianOrc
Copy link
Contributor

What does this PR do?

Adds documentation about the new Python sandbox setting import_notification_policy that was added in temporalio/sdk-python#1187.

Notes to reviewers

@VegetarianOrc VegetarianOrc requested a review from a team as a code owner November 3, 2025 22:58
@vercel
Copy link

vercel bot commented Nov 3, 2025

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Preview Comments Updated (UTC)
temporal-documentation Ready Ready Preview Comment Nov 10, 2025 8:38pm

@CLAassistant
Copy link

CLAassistant commented Nov 3, 2025

CLA assistant check
All committers have signed the CLA.

@github-actions
Copy link
Contributor

github-actions bot commented Nov 3, 2025

📖 Docs PR preview links

@jsundai jsundai self-assigned this Nov 4, 2025
...,
workflow_runner=SandboxedWorkflowRunner(
restrictions=SandboxRestrictions.default.with_import_notification_policy(
workflow.SandboxImportNotificationPolicy.WARN_ON_UNINTENTIONAL_PASSTHROUGH | workflow.SandboxImportNotificationPolicy.WARN_ON_UNINTENTIONAL_PASSTHROUGH
Copy link
Contributor

Choose a reason for hiding this comment

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

This appears twice or maybe it's intentional?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

It wasn't intentional! Thanks for catching. I've updated the example to use the two warning types instead of the same one twice.

Copy link
Contributor

Choose a reason for hiding this comment

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

Ok perfect. Taking a look at this today.

@jsundai jsundai merged commit 16e1090 into main Nov 10, 2025
5 checks passed
@jsundai jsundai deleted the python-sandbox-import-notification-policy branch November 10, 2025 20:44
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