Skip to content

Release v20260413#227

Open
chowbao wants to merge 7 commits intomasterfrom
release-v20260413
Open

Release v20260413#227
chowbao wants to merge 7 commits intomasterfrom
release-v20260413

Conversation

@chowbao
Copy link
Copy Markdown
Contributor

@chowbao chowbao commented Apr 10, 2026

Summary

  • Add AMM TVL metrics (liquidity_pools_tvl) and contract_id to tvl_agg mart model
  • Add dependabot.yml for automated dependency version updates
  • Update CI workflows to run on release branches instead of staging
  • Fix pre-commit configuration

Changes

  • models/marts/tvl/tvl_agg.sql — Add liquidity pool TVL aggregation and contract_id field
  • models/marts/tvl/tvl_agg.yml — Add column descriptions for new fields
  • models/docs/marts/tvl/tvl_agg.md — Update documentation
  • .github/dependabot.yml — New Dependabot config for pip and GitHub Actions updates
  • .github/workflows/ — Point CI triggers at release branches

Test plan

  • Verify tvl_agg model builds successfully with new columns
  • Confirm CI workflows trigger correctly on release branches
  • Validate Dependabot config creates version update PRs

@chowbao chowbao requested a review from a team as a code owner April 10, 2026 17:06
Copilot AI review requested due to automatic review settings April 10, 2026 17:06
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This release PR extends the tvl_agg mart to include AMM (liquidity pool) TVL and introduces automation/ops updates (Dependabot + CI triggers) to better support release-branch workflows.

Changes:

  • Add liquidity_pools_tvl into tvl_agg and update TVL docs accordingly.
  • Add a Dependabot configuration for pip and GitHub Actions updates.
  • Update CI workflows to trigger for PRs targeting release branches instead of staging.

Reviewed changes

Copilot reviewed 6 out of 6 changed files in this pull request and generated 1 comment.

Show a summary per file
File Description
models/marts/tvl/tvl_agg.yml Updates model description and adds the new liquidity_pools_tvl column metadata/tests.
models/marts/tvl/tvl_agg.sql Adds AMM TVL aggregation from asset_balances__daily_agg and introduces contract_id in the output.
models/docs/marts/tvl/tvl_agg.md Adds docs for liquidity_pools_tvl and updates total_tvl docs to include liquidity pools.
.github/workflows/project_evaluator_ci.yml Adjusts workflow trigger branches to include release* instead of staging.
.github/workflows/diff-quality.yml Adjusts workflow trigger branches to include release* instead of staging.
.github/dependabot.yml Adds weekly Dependabot updates for pip and GitHub Actions with grouped PRs.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment on lines +62 to +68
- name: liquidity_pools_tvl
description: '{{ doc("liquidity_pools_tvl") }}'
tests:
- stellar_dbt_public.incremental_not_null:
date_column_name: "day"
greater_than_equal_to: "2 day"

Copy link

Copilot AI Apr 10, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

tvl_agg.sql now outputs a contract_id column (via the join to stg_assets), but this schema YAML doesn’t declare/document that column. This will leave the model docs incomplete and may also trigger the schema change monitoring test. Add a contract_id column entry here (likely with {{ doc("contract_id") }} or the appropriate doc) and decide whether it should have a nullability test (it may be legitimately null for some assets).

Copilot uses AI. Check for mistakes.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants