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

Cleanup directories (_trino_upgrade, airdrops, perpetual_trades, gas_fees, labels) #6376

Merged
merged 8 commits into from
Jul 18, 2024

Conversation

0xRobin
Copy link
Collaborator

@0xRobin 0xRobin commented Jul 16, 2024

Cleanup directories (_trino_upgrade, airdrops, perpetual_trades, gas_fees, labels):

  1. removes unused /analyses and /snapshots dirs
  2. removes old /_trino_upgrade testing directory
  3. moves all airdrop_claim models to /_sector/airdrops/
  4. moves all perpatual_trades models to /_sector/perpetual/
  5. moves /labels to /_sector/labels/
  6. moves /gas (fee models) to /_sector/gas/fess/
  7. cleanup of dbt_project.yml for unused paths

These are all non-modifying changes and will help in the final split into the hourly/daily spellbook subprojects.

@0xRobin 0xRobin changed the title Cleanup directories (_trino_upgrade, airdrops) Cleanup directories (_trino_upgrade, airdrops, perpetual_trades, gas_fees, labels) Jul 17, 2024
@0xRobin 0xRobin marked this pull request as ready for review July 17, 2024 09:47
@0xRobin 0xRobin added ready-for-review this PR development is complete, please review dune team created by dune team labels Jul 17, 2024
@0xRobin 0xRobin requested a review from jeff-dude July 17, 2024 09:47
@jeff-dude
Copy link
Member

i love it -- will make next step to move subprojects easy!
digging in now

@jeff-dude jeff-dude self-assigned this Jul 18, 2024
@jeff-dude jeff-dude added in review Assignee is currently reviewing the PR and removed ready-for-review this PR development is complete, please review labels Jul 18, 2024
0xRobin and others added 2 commits July 18, 2024 17:00
…perpetual_protocol_optimism_perpetual_trades.sql

Co-authored-by: jeff-dude <102681548+jeff-dude@users.noreply.github.com>
…_protocol_perpetual_trades.sql

Co-authored-by: jeff-dude <102681548+jeff-dude@users.noreply.github.com>
@jeff-dude jeff-dude added ready-for-merging and removed in review Assignee is currently reviewing the PR labels Jul 18, 2024
@jeff-dude jeff-dude merged commit d502d33 into duneanalytics:main Jul 18, 2024
1 of 2 checks passed
@github-actions github-actions bot locked and limited conversation to collaborators Jul 18, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
dune team created by dune team ready-for-merging
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants