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

chore(weave): Move old build files #2551

Merged
merged 2 commits into from
Oct 2, 2024
Merged

Conversation

andrewtruong
Copy link
Collaborator

@andrewtruong andrewtruong commented Oct 1, 2024

This just moves the old build files to weave_query

Companion:
https://github.com/wandb/core/pull/24371

@circle-job-mirror
Copy link

circle-job-mirror bot commented Oct 1, 2024

@andrewtruong andrewtruong marked this pull request as ready for review October 1, 2024 14:17
@andrewtruong andrewtruong requested a review from a team as a code owner October 1, 2024 14:17
@@ -1,15 +0,0 @@
include requirements.*
Copy link
Collaborator

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?

Copy link
Collaborator Author

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

Copy link
Collaborator

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?

Copy link
Collaborator Author

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.*
Copy link
Collaborator

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?

Copy link
Collaborator Author

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

Copy link
Collaborator

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

@andrewtruong andrewtruong merged commit de83436 into master Oct 2, 2024
77 checks passed
@andrewtruong andrewtruong deleted the andrew/update-build branch October 2, 2024 02:40
@github-actions github-actions bot locked and limited conversation to collaborators Oct 2, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants