-
Notifications
You must be signed in to change notification settings - Fork 72
Introduce UV as a dependency management #630
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
Conversation
2f82e1d to
08f758a
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Introduces main() method to ccm and then common if __name__ == '__main__': to run it
|
@fruch Rebased to the latest version, is there something else that needs to be done to get this merged? |
|
you'll now need to update the sub-module in dtest |
|
We need to revert it, it broke ccm: Only one example: scylladb/java-driver#526 |
|
Could it be caused by rhe changed version, or do you believe there is a bigger problem here? |
I found the root cause, it was incorrect script path specified. I added it to the version fix PR: #655 Could you try if it fixed your problems? |
|
I can confirm it work, like the following with master version the ccm call was broken |
Overview
pyproject.tomlmain()method toccmscriptTesting