Skip to content

Conversation

ajay-sentry
Copy link
Contributor

@ajay-sentry ajay-sentry commented Oct 2, 2025

This PR memoizes a bunch of calculations that are needed on the frontend

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.

Copy link

sentry-io bot commented Oct 2, 2025

Codecov Report

❌ Patch coverage is 0% with 17 lines in your changes missing coverage. Please review.
✅ Project coverage is 98.51%. Comparing base (8816d1e) to head (d592b6f).
⚠️ Report is 1 commits behind head on main.
✅ All tests successful. No failed tests found.

Files with missing lines Patch % Lines
...poPage/shared/components/CommitInfo/CommitInfo.tsx 0.00% 15 Missing and 2 partials ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #3944      +/-   ##
==========================================
- Coverage   98.62%   98.51%   -0.12%     
==========================================
  Files         828      829       +1     
  Lines       15099    15116      +17     
  Branches     4326     4324       -2     
==========================================
  Hits        14891    14891              
- Misses        200      215      +15     
- Partials        8       10       +2     
Files with missing lines Coverage Δ
...poPage/shared/components/CommitInfo/CommitInfo.tsx 0.00% <0.00%> (ø)
Components Coverage Δ
Assets 100.00% <ø> (ø)
Layouts 99.71% <ø> (ø)
Pages 97.93% <0.00%> (-0.20%) ⬇️
Services 99.32% <ø> (ø)
Shared 99.12% <ø> (ø)
UI 99.01% <ø> (ø)

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 8816d1e...d592b6f. Read the comment docs.

@codecov-staging
Copy link

Bundle Report

Bundle size has no change ✅

@codecov-staging
Copy link

codecov-staging bot commented Oct 2, 2025

Codecov Report

❌ Patch coverage is 0% with 17 lines in your changes missing coverage. Please review.
✅ All tests successful. No failed tests found.

Files with missing lines Patch % Lines
...poPage/shared/components/CommitInfo/CommitInfo.tsx 0.00% 15 Missing and 2 partials ⚠️
@@            Coverage Diff             @@
##             main    #3944      +/-   ##
==========================================
- Coverage   98.62%   98.51%   -0.12%     
==========================================
  Files         828      829       +1     
  Lines       15099    15116      +17     
  Branches     4326     4332       +6     
==========================================
  Hits        14891    14891              
- Misses        200      215      +15     
- Partials        8       10       +2     
Files with missing lines Coverage Δ
...poPage/shared/components/CommitInfo/CommitInfo.tsx 0.00% <0.00%> (ø)
Components Coverage Δ
Assets 100.00% <ø> (ø)
Layouts 99.71% <ø> (ø)
Pages 97.93% <0.00%> (-0.20%) ⬇️
Services 99.32% <ø> (ø)
Shared 99.12% <ø> (ø)
UI 99.01% <ø> (ø)

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 8816d1e...d592b6f. Read the comment docs.

Copy link

codecov-public-qa bot commented Oct 2, 2025

Codecov Report

Attention: Patch coverage is 0% with 17 lines in your changes missing coverage. Please review.

Project coverage is 98.51%. Comparing base (8816d1e) to head (d592b6f).

✅ All tests successful. No failed tests found.

Files with missing lines Patch % Lines
...poPage/shared/components/CommitInfo/CommitInfo.tsx 0.00% 15 Missing and 2 partials ⚠️
@@            Coverage Diff             @@
##             main    #3944      +/-   ##
==========================================
- Coverage   98.62%   98.51%   -0.12%     
==========================================
  Files         828      829       +1     
  Lines       15099    15116      +17     
  Branches     4318     4332      +14     
==========================================
  Hits        14891    14891              
- Misses        200      215      +15     
- Partials        8       10       +2     
Files with missing lines Coverage Δ
...poPage/shared/components/CommitInfo/CommitInfo.tsx 0.00% <0.00%> (ø)
Components Coverage Δ
Assets 100.00% <ø> (ø)
Layouts 99.71% <ø> (ø)
Pages 97.93% <0.00%> (-0.20%) ⬇️
Services 99.32% <ø> (ø)
Shared 99.12% <ø> (ø)
UI 99.01% <ø> (ø)

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 8816d1e...d592b6f. Read the comment docs.

🚀 New features to boost your workflow:
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

Copy link

sentry-io bot commented Oct 2, 2025

Bundle Report

Bundle size has no change ✅

@codecov-releaser
Copy link
Contributor

✅ Deploy preview for gazebo ready!

Previews expire after 1 month automatically.

Storybook

Commit Created Cloud Enterprise
d592b6f Thu, 02 Oct 2025 21:52:20 GMT Cloud Enterprise

@codecov-qa
Copy link

codecov-qa bot commented Oct 2, 2025

Codecov Report

❌ Patch coverage is 0% with 17 lines in your changes missing coverage. Please review.
✅ Project coverage is 98.51%. Comparing base (8816d1e) to head (d592b6f).

Files with missing lines Patch % Lines
...poPage/shared/components/CommitInfo/CommitInfo.tsx 0.00% 15 Missing and 2 partials ⚠️
@@            Coverage Diff             @@
##             main    #3944      +/-   ##
==========================================
+ Coverage   96.54%   98.51%   +1.96%     
==========================================
  Files         828      829       +1     
  Lines       15099    15116      +17     
  Branches     4318     4332      +14     
==========================================
+ Hits        14578    14891     +313     
+ Misses        466      215     -251     
+ Partials       55       10      -45     
Files with missing lines Coverage Δ
...poPage/shared/components/CommitInfo/CommitInfo.tsx 0.00% <0.00%> (ø)

... and 43 files with indirect coverage changes

Components Coverage Δ
Assets 100.00% <ø> (ø)
Layouts 99.71% <ø> (ø)
Pages 97.93% <0.00%> (-0.20%) ⬇️
Services 99.32% <ø> (ø)
Shared 99.12% <ø> (+1.81%) ⬆️
UI 99.01% <ø> (+20.18%) ⬆️

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 8816d1e...d592b6f. Read the comment docs.

🚀 New features to boost your workflow:
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@ajay-sentry ajay-sentry changed the title [test]: FE6 [FE6]: Add Memoization to components Oct 3, 2025
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.

2 participants