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

[draft] Add grouped provider download button #3533

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

rohitvinnakota-codecov
Copy link
Contributor

Description

Code Example

Notable Changes

Screenshots

Link to Sample Entry

Legal Boilerplate

Look, I get it. The entity doing business as "Sentry" was incorporated in the State of Delaware in 2015 as Functional Software, Inc. In 2022 this entity acquired Codecov and as result Sentry is going to need some rights from me in order to utilize my contributions in this PR. So here's the deal: I retain all rights, title and interest in and to my contributions, and by keeping this boilerplate intact I confirm that Sentry can use, modify, copy, and redistribute my contributions, under Sentry's choice of terms.

@codecov-staging
Copy link

codecov-staging bot commented Nov 27, 2024

Bundle Report

Changes will increase total bundle size by 11.85MB (96.32%) ⬆️⚠️, exceeding the configured threshold of 5%.

Bundle name Size Change
gazebo-staging-array-push* 6.14MB 6.14MB (100%) ⬆️⚠️
gazebo-staging-system-esm* 5.72MB 5.72MB (100%) ⬆️⚠️
gazebo-staging-system 6.12MB 648 bytes (0.01%) ⬆️
gazebo-staging-esm 6.19MB 655 bytes (0.01%) ⬆️

ℹ️ *Bundle size includes cached data from a previous commit

Affected Assets, Files, and Routes:

view changes for bundle: gazebo-staging-system

Assets Changed:

Asset Name Size Change Total Size Change (%)
assets/index-legacy.*.js 648 bytes 114.28kB 0.57%

Files in assets/index-legacy.*.js:

  • ./src/pages/CommitDetailPage/CommitCoverage/UploadsCard/UploadsCard.tsx → Total Size: 12.75kB
view changes for bundle: gazebo-staging-esm

Assets Changed:

Asset Name Size Change Total Size Change (%)
assets/index.*.js -1 bytes 89.35kB -0.0%
assets/index.*.js 657 bytes 125.6kB 0.53%
assets/index.*.js -1 bytes 125.63kB -0.0%

Files in assets/index.*.js:

  • ./src/pages/CommitDetailPage/CommitCoverage/UploadsCard/UploadsCard.tsx → Total Size: 12.75kB

Copy link

codecov bot commented Nov 27, 2024

Bundle Report

Changes will increase total bundle size by 11.85MB (96.32%) ⬆️⚠️, exceeding the configured threshold of 5%.

Bundle name Size Change
gazebo-production-array-push* 6.14MB 6.14MB (100%) ⬆️⚠️
gazebo-production-system 6.12MB 648 bytes (0.01%) ⬆️
gazebo-production-system-esm* 5.72MB 5.72MB (100%) ⬆️⚠️
gazebo-production-esm 6.19MB 655 bytes (0.01%) ⬆️

ℹ️ *Bundle size includes cached data from a previous commit

Affected Assets, Files, and Routes:

view changes for bundle: gazebo-production-esm

Assets Changed:

Asset Name Size Change Total Size Change (%)
assets/index.*.js -1 bytes 125.63kB -0.0%
assets/index.*.js -1 bytes 89.35kB -0.0%
assets/index.*.js 657 bytes 125.6kB 0.53%

Files in assets/index.*.js:

  • ./src/pages/CommitDetailPage/CommitCoverage/UploadsCard/UploadsCard.tsx → Total Size: 12.75kB
view changes for bundle: gazebo-production-system

Assets Changed:

Asset Name Size Change Total Size Change (%)
assets/index-legacy.*.js 648 bytes 114.28kB 0.57%

Files in assets/index-legacy.*.js:

  • ./src/pages/CommitDetailPage/CommitCoverage/UploadsCard/UploadsCard.tsx → Total Size: 12.75kB

@codecov-qa
Copy link

codecov-qa bot commented Nov 27, 2024

Codecov Report

Attention: Patch coverage is 14.28571% with 18 lines in your changes missing coverage. Please review.

Project coverage is 98.78%. Comparing base (b4e0043) to head (50a0569).

✅ All tests successful. No failed tests found.

Files with missing lines Patch % Lines
...ailPage/CommitCoverage/UploadsCard/UploadsCard.tsx 14.28% 15 Missing and 3 partials ⚠️

Impacted file tree graph

@@            Coverage Diff             @@
##             main    #3533      +/-   ##
==========================================
- Coverage   98.90%   98.78%   -0.13%     
==========================================
  Files         806      806              
  Lines       14484    14503      +19     
  Branches     4111     4121      +10     
==========================================
+ Hits        14326    14327       +1     
- Misses        151      166      +15     
- Partials        7       10       +3     
Files with missing lines Coverage Δ
...ailPage/CommitCoverage/UploadsCard/UploadsCard.tsx 82.75% <14.28%> (-15.18%) ⬇️
Components Coverage Δ
Assets 100.00% <ø> (ø)
Layouts 99.71% <ø> (ø)
Pages 98.37% <14.28%> (-0.22%) ⬇️
Services 99.33% <ø> (ø)
Shared 99.33% <ø> (ø)
UI 99.14% <ø> (ø)

Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update b4e0043...50a0569. Read the comment docs.

@codecov-staging
Copy link

codecov-staging bot commented Nov 27, 2024

Codecov Report

Attention: Patch coverage is 14.28571% with 18 lines in your changes missing coverage. Please review.

✅ All tests successful. No failed tests found.

Files with missing lines Patch % Lines
...ailPage/CommitCoverage/UploadsCard/UploadsCard.tsx 14.28% 15 Missing and 3 partials ⚠️
@@            Coverage Diff             @@
##             main    #3533      +/-   ##
==========================================
- Coverage   98.74%   98.62%   -0.12%     
==========================================
  Files         826      826              
  Lines       14955    14974      +19     
  Branches     4269     4272       +3     
==========================================
+ Hits        14767    14768       +1     
- Misses        181      196      +15     
- Partials        7       10       +3     
Files with missing lines Coverage Δ
...ailPage/CommitCoverage/UploadsCard/UploadsCard.tsx 82.75% <14.28%> (-15.18%) ⬇️
Components Coverage Δ
Assets 100.00% <ø> (ø)
Layouts 99.69% <ø> (ø)
Pages 98.10% <14.28%> (-0.22%) ⬇️
Services 99.38% <ø> (ø)
Shared 99.12% <ø> (ø)
UI 99.07% <ø> (ø)

Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update ba49650...95d3b4f. Read the comment docs.

🚀 New features to boost your workflow:
  • Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

Copy link

❌ 1 Tests Failed:

Tests completed Failed Passed Skipped
3528 1 3527 0
View the top 1 failed tests by shortest run time
src/pages/PlanPage/subRoutes/UpgradePlanPage/UpgradeForm/Controllers/TeamPlanController/BillingOptions/BillingOptions.test.tsx > BillingOptions &gt; when rendered &gt; planString is set to annual plan &gt; renders annual button as &quot;selected&quot;
Stack Traces | 0.253s run time
Error: expect(element).toHaveClass(&quot;bg-ds-primary-base&quot;)

Expected the element to have class:
  bg-ds-primary-base
Received:
  flex-1 py-1 px-2 text-sm cursor-pointer whitespace-nowrap disabled:text-ds-gray-quaternary disabled:border-ds-gray-tertiary disabled:bg-ds-gray-primary rounded-l
 ❯ .../TeamPlanController/BillingOptions/BillingOptions.test.tsx:127:27

To view individual test run time comparison to the main branch, go to the Test Analytics Dashboard

Copy link

codecov bot commented Nov 27, 2024

Codecov Report

Attention: Patch coverage is 14.28571% with 18 lines in your changes missing coverage. Please review.

Project coverage is 98.78%. Comparing base (b4e0043) to head (50a0569).
Report is 39 commits behind head on main.

✅ All tests successful. No failed tests found.

Files with missing lines Patch % Lines
...ailPage/CommitCoverage/UploadsCard/UploadsCard.tsx 14.28% 15 Missing and 3 partials ⚠️
Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##             main    #3533      +/-   ##
==========================================
- Coverage   98.90%   98.78%   -0.13%     
==========================================
  Files         806      806              
  Lines       14484    14503      +19     
  Branches     4111     4114       +3     
==========================================
+ Hits        14326    14327       +1     
- Misses        151      166      +15     
- Partials        7       10       +3     
Files with missing lines Coverage Δ
...ailPage/CommitCoverage/UploadsCard/UploadsCard.tsx 82.75% <14.28%> (-15.18%) ⬇️
Components Coverage Δ
Assets 100.00% <ø> (ø)
Layouts 99.71% <ø> (ø)
Pages 98.37% <14.28%> (-0.22%) ⬇️
Services 99.33% <ø> (ø)
Shared 99.33% <ø> (ø)
UI 99.14% <ø> (ø)

Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update b4e0043...50a0569. Read the comment docs.

@codecov-releaser
Copy link
Contributor

codecov-releaser commented Nov 27, 2024

✅ Deploy preview for gazebo ready!

Previews expire after 1 month automatically.

Storybook

Commit Created Cloud Enterprise
cd647d2 Wed, 27 Nov 2024 17:09:29 GMT Expired Expired
cd647d2 Wed, 27 Nov 2024 17:09:32 GMT Expired Expired
50a0569 Mon, 09 Dec 2024 17:00:32 GMT Expired Expired
95d3b4f Wed, 05 Mar 2025 20:30:51 GMT Cloud Enterprise

@rohitvinnakota-codecov rohitvinnakota-codecov marked this pull request as ready for review December 11, 2024 18:16
@aj-codecov
Copy link
Contributor

@codecov-ai-reviewer review

1 similar comment
@rohitvinnakota-codecov
Copy link
Contributor Author

@codecov-ai-reviewer review

Copy link
Contributor

Sentry has generated a new PR with unit tests for this PR. View the new PR(#3568) to review the changes.

@sentry-autofix sentry-autofix bot mentioned this pull request Dec 11, 2024
@rohitvinnakota-codecov
Copy link
Contributor Author

@codecov-ai-reviewer review

@rohitvinnakota-codecov
Copy link
Contributor Author

Sentry has generated a new PR with unit tests for this PR. View the new PR(#3795) to review the changes.

@rohitvinnakota-codecov
Copy link
Contributor Author

Sentry has generated a new PR with unit tests for this PR. View the new PR(#3798) to review the changes.

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.

3 participants