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.

Sign up for GitHub

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

Jump to bottom

Budgets #1708

Open
wants to merge 6 commits into
base: master
Choose a base branch
from
Open

Budgets #1708

wants to merge 6 commits into from

Conversation

vio
Copy link
Member

@vio vio commented Nov 16, 2021 •
edited
Loading

rel: #385

@relativeci
Copy link

relativeci bot commented Nov 16, 2021 •
edited
Loading

Job #7554: Bundle Size — 292.6KiB (+0.01%).

d84e24c(current) vs 204e9b5 master#7541(baseline)

Metrics (3 changes)
                 Current
Job #7554		      Baseline
Job #7541
Initial JS 260.94KiB(+0.02%) 260.9KiB
Initial CSS 31.66KiB 31.66KiB
Cache Invalidation 89.18% 0%
Chunks 3 3
Assets 4 4
Modules 557(+0.18%) 556
Duplicate Modules 0 0
Duplicate Code 0% 0%
Packages 27 27
Duplicate Packages 0 0
Total size by type (1 change)
                 Current
Job #7554		      Baseline
Job #7541
CSS 31.66KiB 31.66KiB
Fonts 0B 0B
HTML 0B 0B
IMG 0B 0B
JS 260.94KiB (+0.02%) 260.9KiB
Media 0B 0B
Other 0B 0B

View job #7554 report View budgets branch activity

@lgtm-com
Copy link

lgtm-com bot commented Nov 16, 2021

This pull request introduces 1 alert when merging 65dc741 into 1c6ff44 - view on LGTM.com

new alerts:

  • 1 for Unused variable, import, function or class

@lgtm-com
Copy link

lgtm-com bot commented Nov 17, 2021

This pull request introduces 1 alert when merging f7b8643 into daa447e - view on LGTM.com

new alerts:

  • 1 for Unused variable, import, function or class

@vio vio force-pushed the budgets branch 7 times, most recently from dfa9c1e to 2716865 Compare November 28, 2021 11:36
@vio vio force-pushed the budgets branch 3 times, most recently from 5a428f5 to 72d355e Compare December 14, 2021 20:48
@lgtm-com
Copy link

lgtm-com bot commented Dec 14, 2021

This pull request introduces 1 alert when merging 9c8c86f into e9caf65 - view on LGTM.com

new alerts:

  • 1 for Unused variable, import, function or class

@vio vio force-pushed the budgets branch 3 times, most recently from 1be4ff0 to 7049e47 Compare December 20, 2021 17:26
@vio vio force-pushed the budgets branch 2 times, most recently from 2a7b0e4 to 6297c57 Compare January 18, 2022 21:53
@vio vio force-pushed the budgets branch 2 times, most recently from 9a11f3b to 8d669e5 Compare January 29, 2022 18:35
@vio vio force-pushed the budgets branch 2 times, most recently from ba22de1 to bd5cfc7 Compare February 20, 2022 13:17
@vio vio force-pushed the budgets branch 4 times, most recently from d11770e to 0af9efc Compare March 10, 2022 00:16
@lgtm-com
Copy link

lgtm-com bot commented Mar 12, 2022

This pull request introduces 1 alert when merging b7353c5 into f2fe3a8 - view on LGTM.com

new alerts:

  • 1 for Unused variable, import, function or class

@vio vio force-pushed the budgets branch 3 times, most recently from 1108ae0 to 1aa1662 Compare March 13, 2022 10:48
@lgtm-com
Copy link

lgtm-com bot commented Mar 31, 2022

This pull request introduces 1 alert when merging fb30045 into 941a0b7 - view on LGTM.com

new alerts:

  • 1 for Unused variable, import, function or class

@lgtm-com
Copy link

lgtm-com bot commented Apr 3, 2022

This pull request introduces 1 alert when merging 9c540bd into 6e6561a - view on LGTM.com

new alerts:

  • 1 for Unused variable, import, function or class

@lgtm-com
Copy link

lgtm-com bot commented Apr 9, 2022

This pull request introduces 1 alert when merging 0efdc60 into 6e6561a - view on LGTM.com

new alerts:

  • 1 for Unused variable, import, function or class

@lgtm-com
Copy link

lgtm-com bot commented Apr 9, 2022

This pull request introduces 1 alert when merging 7a66a1e into b9bc517 - view on LGTM.com

new alerts:

  • 1 for Unused variable, import, function or class

@lgtm-com
Copy link

lgtm-com bot commented Apr 10, 2022

This pull request introduces 1 alert when merging 537262b into b9bc517 - view on LGTM.com

new alerts:

  • 1 for Unused variable, import, function or class

@vio vio mentioned this pull request Jan 4, 2023
Closed
github-advanced-security[bot]
github-advanced-security bot found potential problems Jan 4, 2023
View reviewed changes
packages/ui/src/components/budgets/budgets.tsx
Comment on lines +2 to +9
import {
BudgetEvaluated,
BudgetResult,
BudgetStatus,
ConditionOperator,
MetricRunInfoDeltaType,
getGlobalMetricType,
} from '@bundle-stats/utils';

Check notice

Code scanning / CodeQL

Unused variable, import, function or class

Unused import BudgetStatus.
@vio vio force-pushed the budgets branch 2 times, most recently from 6cb2f0c to 7098fc8 Compare January 6, 2023 21:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@vio