Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

♻️ Migrate Pydantic Extra Types to use uv #241

Merged
merged 10 commits into from
Nov 26, 2024
Merged

♻️ Migrate Pydantic Extra Types to use uv #241

merged 10 commits into from
Nov 26, 2024

Conversation

yezz123
Copy link
Collaborator

@yezz123 yezz123 commented Nov 20, 2024

No description provided.

@yezz123 yezz123 self-assigned this Nov 21, 2024
@yezz123 yezz123 added enhancement New feature or request dependencies Pull requests that update a dependency file github_actions Pull requests that update GitHub Actions code labels Nov 21, 2024
samuelcolvin
samuelcolvin reviewed Nov 24, 2024
View reviewed changes
Copy link
Member

@samuelcolvin samuelcolvin left a comment

Choose a reason for hiding this comment

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

otherwise I think this looks really good. 👍

.github/workflows/ci.yml Outdated Show resolved Hide resolved
.github/workflows/ci.yml Outdated Show resolved Hide resolved
.github/workflows/ci.yml Outdated Show resolved Hide resolved
pyproject.toml Outdated Show resolved Hide resolved
pyproject.toml Outdated Show resolved Hide resolved
@yezz123 yezz123 requested a review from samuelcolvin November 24, 2024 22:07
@yezz123 yezz123 linked an issue Nov 24, 2024 that may be closed by this pull request
Migrate to use UV for dependencies management #221
Closed
samuelcolvin
samuelcolvin approved these changes Nov 26, 2024
View reviewed changes
Copy link
Member

@samuelcolvin samuelcolvin left a comment

Choose a reason for hiding this comment

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

Lgtm

@yezz123 yezz123 merged commit b327d7e into main Nov 26, 2024
10 checks passed
@yezz123 yezz123 deleted the uv branch November 26, 2024 11:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@samuelcolvin samuelcolvin samuelcolvin approved these changes

@sydney-runkle sydney-runkle Awaiting requested review from sydney-runkle

Assignees

@yezz123 yezz123

Labels
dependencies Pull requests that update a dependency file enhancement New feature or request github_actions Pull requests that update GitHub Actions code
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Migrate to use UV for dependencies management
3 participants
@yezz123 @samuelcolvin @commonism