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

[Demo] Add number of paying methods to the customers model #9

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

wcchang1115
Copy link
Collaborator

@wcchang1115 wcchang1115 commented Dec 30, 2022

This is a PR that demonstrates how PipeRider Compare Action work.
The workflows attach what has been changed by the new PR model.

@github-actions

This comment was marked as outdated.

@github-actions

This comment was marked as outdated.

@wcchang1115 wcchang1115 marked this pull request as ready for review December 30, 2022 03:02
@wcchang1115 wcchang1115 force-pushed the demo/add-paying-method-count branch from f33fece to 1ea688f Compare July 26, 2023 03:06
@github-actions

This comment was marked as outdated.

@github-actions

This comment was marked as outdated.

@github-actions

This comment was marked as outdated.

@github-actions

This comment was marked as outdated.

@github-actions
Copy link

📊 Piperider Comparison Summary

Impact Summary

PipeRider Report

Code Changes

Added Removed Modified
0 0 1

Resource Impact

Potentially Impacted Assessed Impacted
6 6 assessed, 0 skipped 2

Resource Impact

Models

    Model Impact Columns
Rows Dbt Time Failed Tests All Tests
..mer_order_history_joined.sql Impacted 8 ($\color{green}{\text{ 1 }}$ / $\color{red}{\text{ 0 }}$ / $\color{orange}{\text{ 0 }}$) 100 0:00:00.08 $\color{red}{\text{ (↑ 0.01) }}$ 0 3
models/marts/orders.sql Impacted 16 ($\color{green}{\text{ 1 }}$ / $\color{red}{\text{ 0 }}$ / $\color{orange}{\text{ 0 }}$) 99 0:00:00.05 $\color{red}{\text{ (↑ 0.00) }}$ 0 9

Metrics

    Metric Impact Queries
total ($\color{orange}{\text{ change }}$)
Expenses Assessed not impacted 4
Average Order Amount Assessed not impacted 3
Revenue Assessed not impacted 4
Profit Assessed not impacted 4

📎 Generated Comparison Report ZIP

Find it in the Github Action Runs Page

@github-actions

This comment was marked as outdated.

@wcchang1115 wcchang1115 force-pushed the demo/add-paying-method-count branch from 7610fb0 to 45af07a Compare September 7, 2023 08:07
@github-actions
Copy link

github-actions bot commented Sep 7, 2023

📊 Piperider Comparison Summary

Impact Summary

PipeRider Report

Code Changes

Added Removed Modified
0 0 1

Resource Impact

Potentially Impacted Assessed Impacted
2 2 assessed, 0 skipped 2

Resource Impact

Models

    Model Impact Columns
Rows Dbt Time Failed Tests All Tests
..mer_order_history_joined.sql Impacted 8 ($\color{green}{\text{ 1 }}$ / $\color{red}{\text{ 0 }}$ / $\color{orange}{\text{ 0 }}$) 100 0:00:00.06 $\color{red}{\text{ (↑ 0.00) }}$ 0 3
models/marts/orders.sql Impacted 16 ($\color{green}{\text{ 1 }}$ / $\color{red}{\text{ 0 }}$ / $\color{orange}{\text{ 0 }}$) 99 0:00:00.06 $\color{red}{\text{ (↑ 0.01) }}$ 0 9

Metrics

No changes detected

📎 Generated Comparison Report ZIP

Find it in the Github Action Runs Page

@github-actions
Copy link

📊 Piperider Comparison Summary

Impact Summary

PipeRider Report

Code Changes

Added Removed Modified
0 0 1

Resource Impact

Potentially Impacted Assessed Impacted
2 2 assessed, 0 skipped 2

Resource Impact

Models

    Model Impact Columns
Rows Dbt Time Failed Tests All Tests
..mer_order_history_joined.sql Impacted 8 ($\color{green}{\text{ 1 }}$ / $\color{red}{\text{ 0 }}$ / $\color{orange}{\text{ 0 }}$) 100 0:00:00.19 $\color{red}{\text{ (↑ 0.13) }}$ 0 3
models/marts/orders.sql Impacted 16 ($\color{green}{\text{ 1 }}$ / $\color{red}{\text{ 0 }}$ / $\color{orange}{\text{ 0 }}$) 99 0:00:00.07 $\color{red}{\text{ (↑ 0.00) }}$ 0 9

Metrics

No changes detected

📎 Generated Comparison Report ZIP

Find it in the Github Action Runs Page

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.

1 participant