restruct sentry proj

[bkodes]

The code in /sentry currently does not run.

Cannot access http.client.IncompleteRead.__mro_entries__ from inside a workflow. If this is code from a module not used in a workflow or known to only be us
ed deterministically from a workflow, mark the import as pass through.

Separating the workflow code into its own file resolves the issue

What was changed

Moved greetings workflow to workflows.py and import workflow within worker

Why?

So others can reference this sample and have no issue running it without alterations being made

Checklist

1. Closes

2. How was this tested:

Followed run instructions in README.md. Outputs:

cmd: poetry run python starter.py
output: Workflow result: Hello, World!

3. Any docs updates needed?

No

[CLAassistant]

All committers have signed the CLA.

[cretz]

Thanks! Moving workflows to a separate file is definitely a good thing these days. Just need to get that activity back in there and I think we'll be good.

[cretz]

Please make sure you run poe lint before pushing (poe format will fix the formatting issues)

[cretz]

I think you should update the starter.py to reference the workflow from this file and not through the worker.py file

[brianstrauch]

The restructure you proposed here has since landed in main — the sentry sample now has sentry/workflow.py and sentry/activity.py as separate modules imported by sentry/worker.py, and the sandbox import error is handled explicitly with workflow.unsafe.imports_passed_through(). Most of that came in via the Sentry V2 SDK rework (#140) and follow-up refactors.

Since the change is no longer applicable to the current sample, I'm closing this out.