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

BREAKING: [WIP] UV initial support #712

Draft
wants to merge 4 commits into
base: release/v4.1.0
Choose a base branch
from

Conversation

wahmedswl
Copy link

No description provided.

@surapuramakhil surapuramakhil marked this pull request as draft November 4, 2024 02:36
@surapuramakhil surapuramakhil changed the base branch from main to release/v11.15.2024 November 4, 2024 02:36
@surapuramakhil
Copy link
Collaborator

@wahmedswl can you attach issue you are working on, it there isn't one kindly create it.

@wahmedswl
Copy link
Author

@surapuramakhil #740 is created. UV will help increasing developer happiness and CI/CD times as it's much faster than PIP.

@surapuramakhil
Copy link
Collaborator

surapuramakhil commented Nov 11, 2024

@wahmedswl Are you working on this PR? you also need to take care of

  • updating GitHub actions. CI/CD
  • updating readME

@wahmedswl
Copy link
Author

@surapuramakhil Yes, i got distracted. That make sense; i will update readme and CI/CD scripts as well. I have tried to run it with UV and it seems to work. If test cases are passing, we are good?

Thanks

@surapuramakhil
Copy link
Collaborator

@surapuramakhil Yes, i got distracted. That make sense; i will update readme and CI/CD scripts as well. I have tried to run it with UV and it seems to work. If test cases are passing, we are good?

Thanks

Test cases aren't strong yet.

here is the checklist

  • take project from scratch
  • install all required dependencies using UV
  • run project - make sure code runs through entire critical workflows - at least submitting one or two applications
  • then update ReadME, CI/CD pipelines

If PR is ready, then change it from draft state to open state.

@surapuramakhil surapuramakhil changed the base branch from release/v11.15.2024 to release/v4.1.0 November 19, 2024 04:13
@feder-cr
Copy link
Collaborator

@wahmedswl hi, any updates?

@surapuramakhil surapuramakhil removed their request for review November 20, 2024 21:43
@feder-cr
Copy link
Collaborator

@wahmedswl ?

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