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.

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

Use uv for GitPod, speedup the setup time #1972

Merged
merged 2 commits into from
Jul 8, 2024
Merged

Use uv for GitPod, speedup the setup time #1972

merged 2 commits into from
Jul 8, 2024

Conversation

noklam
Copy link
Contributor

@noklam noklam commented Jul 8, 2024

Description

I discover the GitPod setup is very slow while reviewing some PRs. kedro has switched to uv a few months ago. It's mostly a drop in replacement for pip.

Development notes

Use uv to replace pip, GitPod setup is very slow at installing packages. This is now 10x faster.

QA notes

Checklist

  • Read the contributing guidelines
  • Opened this PR as a 'Draft Pull Request' if it is work-in-progress
  • Updated the documentation to reflect the code changes
  • Added new entries to the RELEASE.md file
  • Added tests to cover my changes

Signed-off-by: Nok Lam Chan <[email protected]>
@noklam noklam requested review from Huongg, rashidakanchwala and ravi-kumar-pilla and removed request for rashidakanchwala July 8, 2024 11:06
@noklam noklam self-assigned this Jul 8, 2024
Copy link
Contributor

@rashidakanchwala rashidakanchwala left a comment

Choose a reason for hiding this comment

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

Look's good. excited for this. thanks @noklam

Copy link
Contributor

@Huongg Huongg left a comment

Choose a reason for hiding this comment

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

amazing, thank you Nok. I also recently used uv thanks to @astrojuanlu

Signed-off-by: Nok <[email protected]>
@noklam noklam enabled auto-merge (squash) July 8, 2024 23:03
@noklam noklam merged commit d157f82 into main Jul 8, 2024
5 checks passed
@noklam noklam deleted the noklam/uv branch July 8, 2024 23:12
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.

3 participants