Skip to content

Conversation

@CasperGN
Copy link
Contributor

@CasperGN CasperGN commented Jan 13, 2026

Description

Changed quickstarts to use uv to conform with main pkg.

Issue reference

We strive to have all PR being opened based on an issue, where the problem or feature have been discussed prior to implementation.

Please reference the issue this PR closes: #146

Checklist

Please make sure you've completed the relevant tasks for this PR, out of the following list:

  • Created/updated tests
  • Tested this change against all the quickstarts
  • Extended the documentation

Note: We expect contributors to open a corresponding documentation PR in the dapr/docs repository. As the implementer, you are the best person to document your work! Implementation PRs will not be merged until the documentation PR is opened and ready for review.

@CasperGN CasperGN requested review from a team as code owners January 13, 2026 13:39
Copy link
Collaborator

@sicoyle sicoyle left a comment

Choose a reason for hiding this comment

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

Thank you! A few comments so far: could you please update the dev docs to reflect the new flow, instead of updating requirements.txt like we used to (uncommenting # -e ../../)? It would be great to document the new uv flow for local package development. 🙏

CasperGN and others added 8 commits January 13, 2026 15:46
Co-authored-by: Sam <[email protected]>
Signed-off-by: Casper Nielsen <[email protected]>
…only optionally used by consumers - not the sdk

Signed-off-by: Casper Nielsen <[email protected]>
…ore (since there's no chromadb packages for 3.14 yet)

Signed-off-by: Casper Nielsen <[email protected]>
@CasperGN
Copy link
Contributor Author

@sicoyle as we agreed I've returned the call to os._exit(0) as per this ref:

A call to sys.exit() is translated into an exception so that clean-up handlers (finally clauses of try statements) can be executed, and so that a debugger can execute a script without running the risk of losing control. The os._exit() function can be used if it is absolutely positively necessary to exit immediately (for example, in the child process after a call to os.fork()).

This is a temporary way to handle exit with the grpc instrumentor while we debug and fix properly closing the channels.
The Quickstarts now run to completion 🚀

@CasperGN
Copy link
Contributor Author

@sicoyle let's wait with merging this until:
dapr/python-sdk#889
dapr/durabletask-python#39

Are merged and new versions created

@sicoyle sicoyle merged commit 321cda7 into dapr:main Jan 16, 2026
6 checks passed
@sicoyle sicoyle deleted the fix(146)/uv-pkg-manager branch January 16, 2026 14:17
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.

python package management in all quickstarts/docs should be consistent

2 participants