-
Notifications
You must be signed in to change notification settings - Fork 73
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
chore(weave): Move old build files #2551
Conversation
Preview this PR with FeatureBee: https://beta.wandb.ai/?betaVersion=924fc8f715859744f5109dde3366375ae056aae3 |
@@ -1,15 +0,0 @@ | |||
include requirements.* |
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.
Are you sure we are good to delete this now? won't this result in build usues?
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.
nah you don't need it, we use pyproject.toml
now
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.
maybe removing the custom builder solves for this?
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.
Yeah you don't need it.
Try uv build
at the repo root
@@ -0,0 +1,6 @@ | |||
include requirements.* |
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.
Why are we putting this here? weave_query is not a python package that gets built, right?
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.
We could if you are confident we will never build this pkg again. You can always go into git history to find this file later if you need it.
Then instead of move, these will be deletes
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.
i see, probably safe to keep
This just moves the old build files to
weave_query
Companion:
https://github.com/wandb/core/pull/24371