Skip to content

Move packages to ./packages/#4

Merged
lesnik512 merged 16 commits intomodern-python:mainfrom
vrslev:uv-workspaces
Nov 5, 2024
Merged

Move packages to ./packages/#4
lesnik512 merged 16 commits intomodern-python:mainfrom
vrslev:uv-workspaces

Conversation

@vrslev
Copy link
Copy Markdown
Collaborator

@vrslev vrslev commented Nov 2, 2024 •
edited
Loading

  • Moved package to packages/ directory
  • Had to rename tests into tests-core and tests-fastapi

Closes #3

@vrslev
Copy link
Copy Markdown
Collaborator Author

vrslev commented Nov 2, 2024

#3 (comment)

This one will be fixed once new uv version is released: astral-sh/uv#6935

@vrslev vrslev changed the title Utilize uv workspaces fully Move packages to packages/ Nov 2, 2024
@vrslev vrslev changed the title Move packages to packages/ Move packages to ./packages/ Nov 2, 2024
@vrslev
Copy link
Copy Markdown
Collaborator Author

vrslev commented Nov 2, 2024

I will mark PR as ready when new uv version is released

@vrslev vrslev requested a review from lesnik512 November 2, 2024 11:01
@vrslev vrslev marked this pull request as ready for review November 5, 2024 07:38
@lesnik512 lesnik512 merged commit d8dfd66 into modern-python:main Nov 5, 2024
@vrslev vrslev deleted the uv-workspaces branch November 5, 2024 10:05
lesnik512 added a commit that referenced this pull request May 1, 2026
@lesnik512
Add cycle detection benchmark and update results
7b3c149 
Measure the per-call overhead of the threading.local cycle guard
(~140 ns/call). Net effect vs original unfixed code is still positive
since fixes #2-#4 saved ~350 ns on the hot path.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@lesnik512 lesnik512 Awaiting requested review from lesnik512

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Try to set up uv workspaces

2 participants

@vrslev @lesnik512