Skip to content

fix: Initial set of parsing errors#3774

Merged
nicholas-codecov merged 4 commits intomainfrom
fix-initial-set-of-parsing-errors
Feb 25, 2025
Merged

fix: Initial set of parsing errors#3774
nicholas-codecov merged 4 commits intomainfrom
fix-initial-set-of-parsing-errors

Conversation

@nicholas-codecov
Copy link
Copy Markdown
Contributor

@nicholas-codecov nicholas-codecov commented Feb 25, 2025 •
edited
Loading

Description

This PR fixes an initial set parsing errors that have been raised in Sentry:

Ticket: codecov/engineering-team#3385

Notable Changes

  • Update schemas for: useRepoBranchContents, SunburstCoverageQueryOpts , useInternalUser, useAccountDetails
  • Update affected things

@codecov
Copy link
Copy Markdown

codecov bot commented Feb 25, 2025 •
edited
Loading

Bundle Report

Changes will increase total bundle size by 70 bytes (0.0%) ⬆️. This is within the configured threshold ✅

Detailed changes
Bundle name Size Change
gazebo-production-system 6.13MB 35 bytes (0.0%) ⬆️
gazebo-production-esm 6.2MB 35 bytes (0.0%) ⬆️

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 630.13kB -0.0%
assets/index.*.js 15 bytes 103.88kB 0.01%
assets/usePrefetchBranchDirEntry.*.js 11 bytes 7.87kB 0.14%
assets/useAccountDetails.*.js 10 bytes 3.52kB 0.28%

Files in assets/index.*.js:

  • ./src/services/user/useInternalUser.ts → Total Size: 1.13kB

Files in assets/index.*.js:

  • ./src/pages/RepoPage/CoverageTab/OverviewTab/subroute/Sunburst/Sunburst.jsx → Total Size: 1.73kB

  • ./src/services/charts/SunburstCoverageQueryOpts.ts → Total Size: 1.29kB

Files in assets/usePrefetchBranchDirEntry.*.js:

  • ./src/services/pathContents/branch/dir/useRepoBranchContents.tsx → Total Size: 4.88kB

Files in assets/useAccountDetails.*.js:

  • ./src/services/account/useAccountDetails.ts → Total Size: 4.69kB
view changes for bundle: gazebo-production-system

Assets Changed:

Asset Name Size Change Total Size Change (%)
assets/index-legacy.*.js -1 bytes 709.73kB -0.0%
assets/index-legacy.*.js 15 bytes 98.32kB 0.02%
assets/usePrefetchBranchDirEntry-legacy.*.js 11 bytes 7.05kB 0.16%
assets/useAccountDetails-legacy.*.js 10 bytes 3.59kB 0.28%

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

  • ./src/services/user/useInternalUser.ts → Total Size: 1.14kB

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

  • ./src/services/charts/SunburstCoverageQueryOpts.ts → Total Size: 1.29kB

  • ./src/pages/RepoPage/CoverageTab/OverviewTab/subroute/Sunburst/Sunburst.jsx → Total Size: 1.73kB

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

  • ./src/services/pathContents/branch/dir/useRepoBranchContents.tsx → Total Size: 4.91kB

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

  • ./src/services/account/useAccountDetails.ts → Total Size: 4.69kB

@codecov-staging
Copy link
Copy Markdown

codecov-staging bot commented Feb 25, 2025

Bundle Report

Changes will increase total bundle size by 70 bytes (0.0%) ⬆️. This is within the configured threshold ✅

Detailed changes
Bundle name Size Change
gazebo-staging-system 6.13MB 35 bytes (0.0%) ⬆️
gazebo-staging-esm 6.2MB 35 bytes (0.0%) ⬆️

Affected Assets, Files, and Routes:

view changes for bundle: gazebo-staging-esm

Assets Changed:

Asset Name Size Change Total Size Change (%)
assets/index.*.js 15 bytes 103.88kB 0.01%
assets/index.*.js -1 bytes 630.13kB -0.0%
assets/usePrefetchBranchDirEntry.*.js 11 bytes 7.87kB 0.14%
assets/useAccountDetails.*.js 10 bytes 3.52kB 0.28%

Files in assets/index.*.js:

  • ./src/services/charts/SunburstCoverageQueryOpts.ts → Total Size: 1.29kB

  • ./src/pages/RepoPage/CoverageTab/OverviewTab/subroute/Sunburst/Sunburst.jsx → Total Size: 1.73kB

Files in assets/index.*.js:

  • ./src/services/user/useInternalUser.ts → Total Size: 1.13kB

Files in assets/usePrefetchBranchDirEntry.*.js:

  • ./src/services/pathContents/branch/dir/useRepoBranchContents.tsx → Total Size: 4.88kB

Files in assets/useAccountDetails.*.js:

  • ./src/services/account/useAccountDetails.ts → Total Size: 4.69kB
view changes for bundle: gazebo-staging-system

Assets Changed:

Asset Name Size Change Total Size Change (%)
assets/index-legacy.*.js 15 bytes 98.32kB 0.02%
assets/index-legacy.*.js -1 bytes 709.73kB -0.0%
assets/usePrefetchBranchDirEntry-legacy.*.js 11 bytes 7.05kB 0.16%
assets/useAccountDetails-legacy.*.js 10 bytes 3.59kB 0.28%

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

  • ./src/services/charts/SunburstCoverageQueryOpts.ts → Total Size: 1.29kB

  • ./src/pages/RepoPage/CoverageTab/OverviewTab/subroute/Sunburst/Sunburst.jsx → Total Size: 1.73kB

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

  • ./src/services/user/useInternalUser.ts → Total Size: 1.14kB

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

  • ./src/services/pathContents/branch/dir/useRepoBranchContents.tsx → Total Size: 4.91kB

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

  • ./src/services/account/useAccountDetails.ts → Total Size: 4.69kB

@codecov-qa
Copy link
Copy Markdown

codecov-qa bot commented Feb 25, 2025 •
edited
Loading

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 98.72%. Comparing base (88989bb) to head (929da0a).
Report is 6 commits behind head on main.

✅ All tests successful. No failed tests found.

@@            Coverage Diff             @@
##             main    #3774      +/-   ##
==========================================
+ Coverage   98.69%   98.72%   +0.02%     
==========================================
  Files         825      824       -1     
  Lines       14826    14782      -44     
  Branches     4242     4247       +5     
==========================================
- Hits        14633    14593      -40     
+ Misses        186      182       -4     
  Partials        7        7
Files with missing lines Coverage Δ
...rageTab/OverviewTab/subroute/Sunburst/Sunburst.jsx 90.47% <100.00%> (ø)
src/services/account/useAccountDetails.ts 100.00% <100.00%> (ø)
src/services/charts/SunburstCoverageQueryOpts.ts 93.75% <100.00%> (ø)
.../pathContents/branch/dir/useRepoBranchContents.tsx 100.00% <ø> (ø)
src/services/user/useInternalUser.ts 100.00% <ø> (ø)

... and 8 files with indirect coverage changes

Components Coverage Δ
Assets 100.00% <ø> (ø)
Layouts 99.69% <ø> (ø)
Pages 98.29% <100.00%> (+0.03%) ⬆️
Services 99.36% <100.00%> (+0.02%) ⬆️
Shared 99.11% <ø> (ø)
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 88989bb...929da0a. Read the comment docs.

@codecov
Copy link
Copy Markdown

codecov bot commented Feb 25, 2025 •
edited
Loading

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 98.72%. Comparing base (88989bb) to head (929da0a).
Report is 6 commits behind head on main.

✅ All tests successful. No failed tests found.

Additional details and impacted files 
@@            Coverage Diff             @@
##             main    #3774      +/-   ##
==========================================
+ Coverage   98.69%   98.72%   +0.02%     
==========================================
  Files         825      824       -1     
  Lines       14826    14782      -44     
  Branches     4242     4239       -3     
==========================================
- Hits        14633    14593      -40     
+ Misses        186      182       -4     
  Partials        7        7
Files with missing lines Coverage Δ
...rageTab/OverviewTab/subroute/Sunburst/Sunburst.jsx 90.47% <100.00%> (ø)
src/services/account/useAccountDetails.ts 100.00% <100.00%> (ø)
src/services/charts/SunburstCoverageQueryOpts.ts 93.75% <100.00%> (ø)
.../pathContents/branch/dir/useRepoBranchContents.tsx 100.00% <ø> (ø)
src/services/user/useInternalUser.ts 100.00% <ø> (ø)

... and 8 files with indirect coverage changes

Components Coverage Δ
Assets 100.00% <ø> (ø)
Layouts 99.69% <ø> (ø)
Pages 98.29% <100.00%> (+0.03%) ⬆️
Services 99.36% <100.00%> (+0.02%) ⬆️
Shared 99.11% <ø> (ø)
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 88989bb...929da0a. Read the comment docs.

@codecov-notifications
Copy link
Copy Markdown

codecov-notifications bot commented Feb 25, 2025 •
edited
Loading

Codecov Report

All modified and coverable lines are covered by tests ✅

✅ All tests successful. No failed tests found.

@@            Coverage Diff             @@
##             main    #3774      +/-   ##
==========================================
+ Coverage   98.69%   98.72%   +0.02%     
==========================================
  Files         825      824       -1     
  Lines       14826    14782      -44     
  Branches     4250     4247       -3     
==========================================
- Hits        14633    14593      -40     
+ Misses        186      182       -4     
  Partials        7        7
Files with missing lines Coverage Δ
...rageTab/OverviewTab/subroute/Sunburst/Sunburst.jsx 90.47% <100.00%> (ø)
src/services/account/useAccountDetails.ts 100.00% <100.00%> (ø)
src/services/charts/SunburstCoverageQueryOpts.ts 93.75% <100.00%> (ø)
.../pathContents/branch/dir/useRepoBranchContents.tsx 100.00% <ø> (ø)
src/services/user/useInternalUser.ts 100.00% <ø> (ø)

... and 8 files with indirect coverage changes

Components Coverage Δ
Assets 100.00% <ø> (ø)
Layouts 99.69% <ø> (ø)
Pages 98.29% <100.00%> (+0.03%) ⬆️
Services 99.36% <100.00%> (+0.02%) ⬆️
Shared 99.11% <ø> (ø)
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 88989bb...929da0a. Read the comment docs.

@codecov-public-qa
Copy link
Copy Markdown

codecov-public-qa bot commented Feb 25, 2025 •
edited
Loading

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 98.72%. Comparing base (88989bb) to head (929da0a).
Report is 6 commits behind head on main.

✅ All tests successful. No failed tests found.

@@            Coverage Diff             @@
##             main    #3774      +/-   ##
==========================================
+ Coverage   98.69%   98.72%   +0.02%     
==========================================
  Files         825      824       -1     
  Lines       14826    14782      -44     
  Branches     4242     4239       -3     
==========================================
- Hits        14633    14593      -40     
+ Misses        186      182       -4     
  Partials        7        7
Files with missing lines Coverage Δ
...rageTab/OverviewTab/subroute/Sunburst/Sunburst.jsx 90.47% <100.00%> (ø)
src/services/account/useAccountDetails.ts 100.00% <100.00%> (ø)
src/services/charts/SunburstCoverageQueryOpts.ts 93.75% <100.00%> (ø)
.../pathContents/branch/dir/useRepoBranchContents.tsx 100.00% <ø> (ø)
src/services/user/useInternalUser.ts 100.00% <ø> (ø)

... and 8 files with indirect coverage changes

Components Coverage Δ
Assets 100.00% <ø> (ø)
Layouts 99.69% <ø> (ø)
Pages 98.29% <100.00%> (+0.03%) ⬆️
Services 99.36% <100.00%> (+0.02%) ⬆️
Shared 99.11% <ø> (ø)
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 88989bb...929da0a. Read the comment docs.

@codecov-releaser
Copy link
Copy Markdown
Collaborator

codecov-releaser commented Feb 25, 2025

✅ Deploy preview for gazebo ready!

Previews expire after 1 month automatically.

Storybook

Commit Created Cloud Enterprise
929da0a Tue, 25 Feb 2025 17:06:17 GMT Cloud Enterprise

ajay-sentry
ajay-sentry reviewed Feb 25, 2025
View reviewed changes
src/services/pathContents/branch/dir/useRepoBranchContents.test.tsx

beforeEach(() => {
console.error = () => null
vi.spyOn(console, 'error').mockImplementation(() => {})
Copy link
Copy Markdown
Contributor

@ajay-sentry ajay-sentry Feb 25, 2025

Choose a reason for hiding this comment

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

nice

ajay-sentry
ajay-sentry approved these changes Feb 25, 2025
View reviewed changes
Copy link
Copy Markdown
Contributor

@ajay-sentry ajay-sentry left a comment

Choose a reason for hiding this comment

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

lgtm

@nicholas-codecov nicholas-codecov added this pull request to the merge queue Feb 25, 2025
Merged via the queue into main with commit 9136ccd Feb 25, 2025
62 checks passed
@nicholas-codecov nicholas-codecov deleted the fix-initial-set-of-parsing-errors branch February 25, 2025 19:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@ajay-sentry ajay-sentry ajay-sentry approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@nicholas-codecov @codecov-releaser @ajay-sentry @nsdeschenes