chore: Update usePullPageData to TSQ V5 #3536

nicholas-codecov · 2024-11-28T17:48:11Z

Description

This PR updates usePullPageData to PullPageDataQueryOpts as part of our migration over to TS Query V5.

Ticket: codecov/engineering-team#2965

Notable Changes

Update usePullPageData to PullPageDataQueryOpts
Some tidying up of directories, and getting rid of barrel index files
Update tests

codecov-staging · 2024-11-28T17:50:28Z

Bundle Report

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

Detailed changes

Bundle name	Size	Change
gazebo-staging-system-esm	5.72MB	1.11kB (0.02%) ⬆️
gazebo-staging-system	5.67MB	989 bytes (0.02%) ⬆️

codecov · 2024-11-28T17:50:30Z

Bundle Report

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

Detailed changes

Bundle name	Size	Change
gazebo-production-system	5.67MB	989 bytes (0.02%) ⬆️
gazebo-production-system-esm	5.72MB	1.11kB (0.02%) ⬆️

codecov-releaser · 2024-11-28T17:54:06Z

✅ Deploy preview for gazebo ready!

Previews expire after 1 month automatically.

Storybook

Commit	Created	Cloud	Enterprise
`7f58895`	Thu, 28 Nov 2024 17:54:06 GMT	Expired	Expired
`659948b`	Fri, 29 Nov 2024 11:17:08 GMT	Cloud	Enterprise

…tsqv5

codecov-notifications · 2024-11-29T11:15:09Z

Codecov Report

All modified and coverable lines are covered by tests ✅

✅ All tests successful. No failed tests found.

@@           Coverage Diff           @@
##             main    #3536   +/-   ##
=======================================
  Coverage   98.91%   98.92%           
=======================================
  Files         806      806           
  Lines       14443    14447    +4     
  Branches     4097     4105    +8     
=======================================
+ Hits        14287    14291    +4     
  Misses        149      149           
  Partials        7        7

Files with missing lines	Coverage Δ
...uestPage/PullBundleAnalysis/PullBundleAnalysis.tsx	`100.00% <100.00%> (ø)`
...ages/PullRequestPage/PullCoverage/PullCoverage.jsx	`100.00% <100.00%> (ø)`
...PullCoverage/PullCoverageTabs/PullCoverageTabs.jsx	`100.00% <ø> (ø)`
...age/PullCoverage/PullCoverageTabs/useTabsCounts.ts	`100.00% <100.00%> (ø)`
...Page/PullCoverage/routes/FileViewer/FileViewer.jsx	`100.00% <100.00%> (ø)`
src/pages/PullRequestPage/PullRequestPage.tsx	`97.56% <100.00%> (+0.26%)`	⬆️
.../PullRequestPage/queries/PullPageDataQueryOpts.tsx	`100.00% <100.00%> (ø)`

Components	Coverage Δ
Assets	`100.00% <ø> (ø)`
Layouts	`100.00% <ø> (ø)`
Pages	`98.59% <100.00%> (+<0.01%)`	⬆️
Services	`99.33% <ø> (ø)`
Shared	`99.33% <ø> (ø)`
UI	`99.21% <ø> (ø)`

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 b2a584b...659948b. Read the comment docs.

codecov-qa · 2024-11-29T11:15:10Z

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 98.92%. Comparing base (b2a584b) to head (659948b).
Report is 1 commits behind head on main.

✅ All tests successful. No failed tests found.

@@           Coverage Diff           @@
##             main    #3536   +/-   ##
=======================================
  Coverage   98.91%   98.92%           
=======================================
  Files         806      806           
  Lines       14443    14447    +4     
  Branches     4104     4098    -6     
=======================================
+ Hits        14287    14291    +4     
  Misses        149      149           
  Partials        7        7

Files with missing lines	Coverage Δ
...uestPage/PullBundleAnalysis/PullBundleAnalysis.tsx	`100.00% <100.00%> (ø)`
...ages/PullRequestPage/PullCoverage/PullCoverage.jsx	`100.00% <100.00%> (ø)`
...PullCoverage/PullCoverageTabs/PullCoverageTabs.jsx	`100.00% <ø> (ø)`
...age/PullCoverage/PullCoverageTabs/useTabsCounts.ts	`100.00% <100.00%> (ø)`
...Page/PullCoverage/routes/FileViewer/FileViewer.jsx	`100.00% <100.00%> (ø)`
src/pages/PullRequestPage/PullRequestPage.tsx	`97.56% <100.00%> (+0.26%)`	⬆️
.../PullRequestPage/queries/PullPageDataQueryOpts.tsx	`100.00% <100.00%> (ø)`

Components	Coverage Δ
Assets	`100.00% <ø> (ø)`
Layouts	`100.00% <ø> (ø)`
Pages	`98.59% <100.00%> (+<0.01%)`	⬆️
Services	`99.33% <ø> (ø)`
Shared	`99.33% <ø> (ø)`
UI	`99.21% <ø> (ø)`

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 b2a584b...659948b. Read the comment docs.

codecov · 2024-11-29T11:15:21Z

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 98.92%. Comparing base (b2a584b) to head (659948b).
Report is 1 commits behind head on main.

✅ All tests successful. No failed tests found.

Additional details and impacted files

@@           Coverage Diff           @@
##             main    #3536   +/-   ##
=======================================
  Coverage   98.91%   98.92%           
=======================================
  Files         806      806           
  Lines       14443    14447    +4     
  Branches     4097     4098    +1     
=======================================
+ Hits        14287    14291    +4     
  Misses        149      149           
  Partials        7        7

Files with missing lines	Coverage Δ
...uestPage/PullBundleAnalysis/PullBundleAnalysis.tsx	`100.00% <100.00%> (ø)`
...ages/PullRequestPage/PullCoverage/PullCoverage.jsx	`100.00% <100.00%> (ø)`
...PullCoverage/PullCoverageTabs/PullCoverageTabs.jsx	`100.00% <ø> (ø)`
...age/PullCoverage/PullCoverageTabs/useTabsCounts.ts	`100.00% <100.00%> (ø)`
...Page/PullCoverage/routes/FileViewer/FileViewer.jsx	`100.00% <100.00%> (ø)`
src/pages/PullRequestPage/PullRequestPage.tsx	`97.56% <100.00%> (+0.26%)`	⬆️
.../PullRequestPage/queries/PullPageDataQueryOpts.tsx	`100.00% <100.00%> (ø)`

Components	Coverage Δ
Assets	`100.00% <ø> (ø)`
Layouts	`100.00% <ø> (ø)`
Pages	`98.59% <100.00%> (+<0.01%)`	⬆️
Services	`99.33% <ø> (ø)`
Shared	`99.33% <ø> (ø)`
UI	`99.21% <ø> (ø)`

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 b2a584b...659948b. Read the comment docs.

codecov-public-qa · 2024-11-29T13:11:12Z

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 98.92%. Comparing base (b2a584b) to head (659948b).
Report is 1 commits behind head on main.

✅ All tests successful. No failed tests found.

@@           Coverage Diff           @@
##             main    #3536   +/-   ##
=======================================
  Coverage   98.91%   98.92%           
=======================================
  Files         806      806           
  Lines       14443    14447    +4     
  Branches     4104     4098    -6     
=======================================
+ Hits        14287    14291    +4     
  Misses        149      149           
  Partials        7        7

Files with missing lines	Coverage Δ
...uestPage/PullBundleAnalysis/PullBundleAnalysis.tsx	`100.00% <100.00%> (ø)`
...ages/PullRequestPage/PullCoverage/PullCoverage.jsx	`100.00% <100.00%> (ø)`
...PullCoverage/PullCoverageTabs/PullCoverageTabs.jsx	`100.00% <ø> (ø)`
...age/PullCoverage/PullCoverageTabs/useTabsCounts.ts	`100.00% <100.00%> (ø)`
...Page/PullCoverage/routes/FileViewer/FileViewer.jsx	`100.00% <100.00%> (ø)`
src/pages/PullRequestPage/PullRequestPage.tsx	`97.56% <100.00%> (+0.26%)`	⬆️
.../PullRequestPage/queries/PullPageDataQueryOpts.tsx	`100.00% <100.00%> (ø)`

Components	Coverage Δ
Assets	`100.00% <ø> (ø)`
Layouts	`100.00% <ø> (ø)`
Pages	`98.59% <100.00%> (+<0.01%)`	⬆️
Services	`99.33% <ø> (ø)`
Shared	`99.33% <ø> (ø)`
UI	`99.21% <ø> (ø)`

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 b2a584b...659948b. Read the comment docs.

sentry-io · 2024-12-05T07:11:19Z

Suspect Issues

This pull request was deployed and Sentry observed the following issues:

‼️ TypeError: Failed to fetch dynamically imported module: https://app.codecov.io/assets/index.CjjySavq0ebtoC9C... /:provider/:owner/:repo/pull/:pullId View Issue
‼️ TypeError: Failed to fetch /:provider/:owner/:repo/pull/:pullId/blob/:path+ View Issue
‼️ SyntaxError: Unexpected token 'u', "upstream c"... is not valid JSON /:provider/:owner/:repo/pull/:pullId View Issue
‼️ TypeError: NetworkError when attempting to fetch resource. /:provider/:owner/:repo/pull/:pullId View Issue
‼️ TypeError: Failed to fetch /:provider/:owner/:repo/pull/:pullId View Issue

_{Did you find this useful? React with a 👍 or 👎}

nicholas-codecov added 7 commits November 28, 2024 13:41

refactor usePullPageData to PullPageDataQueryOpts

febacd4

update useTabsCounts to use query opts

3893198

update FileViewer to use query opts

00b6ead

update PullCoverageTabs to use new query opts

052b745

update PullCoverage to use query opts

b901ea8

update PullRequestPage to use new query opts

a4f1164

update PullBundleAnalysis to use new query opts

7f58895

nicholas-codecov changed the title ~~chore: update usePullPageData to TSQ V5~~ chore: Update usePullPageData to TSQ V5 Nov 28, 2024

rohitvinnakota-codecov approved these changes Nov 28, 2024

View reviewed changes

nicholas-codecov enabled auto-merge November 29, 2024 11:10

nicholas-codecov disabled auto-merge November 29, 2024 11:11

Merge branch 'main' into gh-eng-2965-chore-update-usePullPageData-to-…

659948b

…tsqv5

nicholas-codecov added this pull request to the merge queue Nov 29, 2024

github-merge-queue bot removed this pull request from the merge queue due to failed status checks Nov 29, 2024

nicholas-codecov added this pull request to the merge queue Nov 29, 2024

Merged via the queue into main with commit 67b950a Nov 29, 2024
59 of 62 checks passed

nicholas-codecov deleted the gh-eng-2965-chore-update-usePullPageData-to-tsqv5 branch November 29, 2024 13:11

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

chore: Update usePullPageData to TSQ V5 #3536

chore: Update usePullPageData to TSQ V5 #3536

nicholas-codecov commented Nov 28, 2024

codecov-staging bot commented Nov 28, 2024 •

edited

Loading

codecov bot commented Nov 28, 2024 •

edited

Loading

codecov-releaser commented Nov 28, 2024 •

edited

Loading

codecov-notifications bot commented Nov 29, 2024 •

edited

Loading

codecov-qa bot commented Nov 29, 2024 •

edited

Loading

codecov bot commented Nov 29, 2024 •

edited

Loading

codecov-public-qa bot commented Nov 29, 2024

sentry-io bot commented Dec 5, 2024 •

edited

Loading

chore: Update usePullPageData to TSQ V5 #3536

chore: Update usePullPageData to TSQ V5 #3536

Conversation

nicholas-codecov commented Nov 28, 2024

Description

Notable Changes

codecov-staging bot commented Nov 28, 2024 • edited Loading

Bundle Report

codecov bot commented Nov 28, 2024 • edited Loading

Bundle Report

codecov-releaser commented Nov 28, 2024 • edited Loading

✅ Deploy preview for gazebo ready!

codecov-notifications bot commented Nov 29, 2024 • edited Loading

Codecov Report

codecov-qa bot commented Nov 29, 2024 • edited Loading

Codecov Report

codecov bot commented Nov 29, 2024 • edited Loading

Codecov Report

codecov-public-qa bot commented Nov 29, 2024

Codecov Report

sentry-io bot commented Dec 5, 2024 • edited Loading

Suspect Issues

codecov-staging bot commented Nov 28, 2024 •

edited

Loading

codecov bot commented Nov 28, 2024 •

edited

Loading

codecov-releaser commented Nov 28, 2024 •

edited

Loading

codecov-notifications bot commented Nov 29, 2024 •

edited

Loading

codecov-qa bot commented Nov 29, 2024 •

edited

Loading

codecov bot commented Nov 29, 2024 •

edited

Loading

sentry-io bot commented Dec 5, 2024 •

edited

Loading