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): Legacy Refactor pt18 #2236

Merged
merged 7 commits into from
Aug 28, 2024
Merged

chore(weave): Legacy Refactor pt18 #2236

merged 7 commits into from
Aug 28, 2024

Conversation

andrewtruong
Copy link
Collaborator

@andrewtruong andrewtruong commented Aug 27, 2024

Split out from: #2182

This slice moves the following to legacy:

  • legacy tests
  • legacy test_utils
  • old query panel examples and notebooks
  • makefile

And deletes:

  • Extraneous files
  • The top level api.py, splitting out the trace and query apis into their respective api.py

@andrewtruong andrewtruong requested a review from a team as a code owner August 27, 2024 04:36
@circle-job-mirror
Copy link

circle-job-mirror bot commented Aug 27, 2024

@andrewtruong andrewtruong merged commit ebc8d91 into andrew/ref17 Aug 28, 2024
6 checks passed
@andrewtruong andrewtruong deleted the andrew/ref18 branch August 28, 2024 19:13
andrewtruong added a commit that referenced this pull request Aug 28, 2024
* move to trace: init_message

* move to trace: weave_init

* move to trace: cli

* move to trace: client_context

* chore(weave): Legacy Refactor pt18 (#2236)

* delete: extraneous files

* delete: api; split out trace and query api from single api file

* move to legacy: legacy tests

* move to legacy: legacy test_utils

* move to legacy: old query panel examples and notebooks

* move to legacy: makefile

* !!! move files from weave/legacy -> weave/legacy/weave to match new repo (#2237)
@github-actions github-actions bot locked and limited conversation to collaborators Aug 28, 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