Skip to content

Conversation

@spalmurray
Copy link
Contributor

@spalmurray spalmurray commented Jul 15, 2025

Changes the header and adds a little tooltip on the flags table because it's confusing.

@codecov-staging
Copy link

codecov-staging bot commented Jul 15, 2025

Bundle Report

Changes will increase total bundle size by 1.56kB (0.01%) ⬆️. This is within the configured threshold ✅

Detailed changes
Bundle name Size Change
gazebo-staging-system 6.23MB 782 bytes (0.01%) ⬆️
gazebo-staging-esm 6.31MB 780 bytes (0.01%) ⬆️

Affected Assets, Files, and Routes:

view changes for bundle: gazebo-staging-esm

Assets Changed:

Asset Name Size Change Total Size Change (%)
assets/index.*.js 780 bytes 90.02kB 0.87%

Files in assets/index.*.js:

  • ./src/pages/RepoPage/CoverageTab/FlagsTab/subroute/FlagsTable/FlagsTable.tsx → Total Size: 9.26kB
view changes for bundle: gazebo-staging-system

Assets Changed:

Asset Name Size Change Total Size Change (%)
assets/index-legacy.*.js 782 bytes 84.77kB 0.93%

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

  • ./src/pages/RepoPage/CoverageTab/FlagsTab/subroute/FlagsTable/FlagsTable.tsx → Total Size: 9.26kB

@codecov
Copy link

codecov bot commented Jul 15, 2025

Bundle Report

Changes will increase total bundle size by 1.56kB (0.01%) ⬆️. This is within the configured threshold ✅

Detailed changes
Bundle name Size Change
gazebo-production-system 6.23MB 782 bytes (0.01%) ⬆️
gazebo-production-esm 6.31MB 780 bytes (0.01%) ⬆️

Affected Assets, Files, and Routes:

view changes for bundle: gazebo-production-esm

Assets Changed:

Asset Name Size Change Total Size Change (%)
assets/index.*.js 780 bytes 90.02kB 0.87%

Files in assets/index.*.js:

  • ./src/pages/RepoPage/CoverageTab/FlagsTab/subroute/FlagsTable/FlagsTable.tsx → Total Size: 9.26kB
view changes for bundle: gazebo-production-system

Assets Changed:

Asset Name Size Change Total Size Change (%)
assets/index-legacy.*.js 782 bytes 84.77kB 0.93%

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

  • ./src/pages/RepoPage/CoverageTab/FlagsTab/subroute/FlagsTable/FlagsTable.tsx → Total Size: 9.26kB

@codecov-releaser
Copy link
Contributor

codecov-releaser commented Jul 15, 2025

✅ Deploy preview for gazebo ready!

Previews expire after 1 month automatically.

Storybook

Commit Created Cloud Enterprise
073809d Tue, 15 Jul 2025 19:07:55 GMT Expired Expired
073809d Tue, 15 Jul 2025 19:09:02 GMT Expired Expired
2c920b1 Tue, 15 Jul 2025 19:35:31 GMT Expired Expired
f297c9b Tue, 15 Jul 2025 20:38:50 GMT Expired Expired
f297c9b Tue, 15 Jul 2025 20:39:11 GMT Expired Expired
efa0405 Wed, 16 Jul 2025 18:28:06 GMT Cloud Enterprise

@codecov
Copy link

codecov bot commented Jul 15, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 98.63%. Comparing base (5f3314c) to head (f297c9b).

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #3912   +/-   ##
=======================================
  Coverage   98.63%   98.63%           
=======================================
  Files         828      828           
  Lines       15092    15092           
  Branches     4314     4322    +8     
=======================================
  Hits        14886    14886           
  Misses        198      198           
  Partials        8        8           
Files with missing lines Coverage Δ
...ageTab/FlagsTab/subroute/FlagsTable/FlagsTable.tsx 100.00% <100.00%> (ø)
Components Coverage Δ
Assets 100.00% <ø> (ø)
Layouts 99.71% <ø> (ø)
Pages 98.15% <100.00%> (ø)
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 5f3314c...f297c9b. Read the comment docs.

@codecov-qa
Copy link

codecov-qa bot commented Jul 15, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 98.63%. Comparing base (5f3314c) to head (f297c9b).

✅ All tests successful. No failed tests found.

@@           Coverage Diff           @@
##             main    #3912   +/-   ##
=======================================
  Coverage   98.63%   98.63%           
=======================================
  Files         828      828           
  Lines       15092    15092           
  Branches     4314     4314           
=======================================
  Hits        14886    14886           
  Misses        198      198           
  Partials        8        8           
Files with missing lines Coverage Δ
...ageTab/FlagsTab/subroute/FlagsTable/FlagsTable.tsx 100.00% <100.00%> (ø)
Components Coverage Δ
Assets 100.00% <ø> (ø)
Layouts 99.71% <ø> (ø)
Pages 98.15% <100.00%> (ø)
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 5f3314c...f297c9b. 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.

@codecov-notifications
Copy link

codecov-notifications bot commented Jul 15, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

✅ All tests successful. No failed tests found.

@@           Coverage Diff           @@
##             main    #3912   +/-   ##
=======================================
  Coverage   98.63%   98.63%           
=======================================
  Files         828      828           
  Lines       15092    15092           
  Branches     4322     4322           
=======================================
  Hits        14886    14886           
  Misses        198      198           
  Partials        8        8           
Files with missing lines Coverage Δ
...ageTab/FlagsTab/subroute/FlagsTable/FlagsTable.tsx 100.00% <100.00%> (ø)
Components Coverage Δ
Assets 100.00% <ø> (ø)
Layouts 99.71% <ø> (ø)
Pages 98.15% <100.00%> (ø)
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 5f3314c...f297c9b. Read the comment docs.

@codecov-public-qa
Copy link

codecov-public-qa bot commented Jul 15, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 98.63%. Comparing base (5f3314c) to head (f297c9b).

✅ All tests successful. No failed tests found.

@@           Coverage Diff           @@
##             main    #3912   +/-   ##
=======================================
  Coverage   98.63%   98.63%           
=======================================
  Files         828      828           
  Lines       15092    15092           
  Branches     4314     4314           
=======================================
  Hits        14886    14886           
  Misses        198      198           
  Partials        8        8           
Files with missing lines Coverage Δ
...ageTab/FlagsTab/subroute/FlagsTable/FlagsTable.tsx 100.00% <100.00%> (ø)
Components Coverage Δ
Assets 100.00% <ø> (ø)
Layouts 99.71% <ø> (ø)
Pages 98.15% <100.00%> (ø)
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 5f3314c...f297c9b. 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.

@spalmurray spalmurray marked this pull request as ready for review July 15, 2025 20:35
@spalmurray spalmurray requested a review from a team July 15, 2025 20:35
>
This is the average flag coverage over the selected time period.
To view current coverage % for a given flag, navigate to the
Overview tab and filter by flag.
Copy link
Contributor

Choose a reason for hiding this comment

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

Optional, not sure if a link to the overview tab would be helpful here

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Probably

Copy link
Contributor

@ElioDiNino ElioDiNino left a comment

Choose a reason for hiding this comment

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

Nice improvement

@ajay-sentry
Copy link
Contributor

bugbot run

@spalmurray
Copy link
Contributor Author

no don't do it bugbot 😲

@ajay-sentry
Copy link
Contributor

cursor review

@ajay-sentry
Copy link
Contributor

cursor review verbose=true

@cursor
Copy link

cursor bot commented Jul 16, 2025

BugBot request id: serverGenReqId_9de67ef1-5fed-427c-9d44-2c54110a0f1d

@spalmurray
Copy link
Contributor Author

closing for Ajay

@spalmurray spalmurray closed this Jul 16, 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.

6 participants