Skip to content

ref: Convert FileEntry to TS#3772

Merged
spalmurray-codecov merged 3 commits intomainfrom
spalmurray/fileentry-ts
Feb 25, 2025
Merged

ref: Convert FileEntry to TS#3772
spalmurray-codecov merged 3 commits intomainfrom
spalmurray/fileentry-ts

Conversation

@spalmurray-codecov
Copy link
Copy Markdown
Contributor

@spalmurray-codecov spalmurray-codecov commented Feb 25, 2025

Converts FileEntry to TSX.

Part of codecov/engineering-team#2747

@codecov-staging
Copy link
Copy Markdown

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

Bundle Report

Changes will decrease total bundle size by 506 bytes (-0.0%) ⬇️. This is within the configured threshold ✅

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

Affected Assets, Files, and Routes:

view changes for bundle: gazebo-staging-system

Assets Changed:

Asset Name Size Change Total Size Change (%)
assets/RawFileViewer-legacy.*.js -259 bytes 127.3kB -0.2%

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

  • ./src/shared/ContentsTable/TableEntries/BaseEntries/FileEntry.tsx → Total Size: 1.25kB
view changes for bundle: gazebo-staging-esm

Assets Changed:

Asset Name Size Change Total Size Change (%)
assets/RawFileViewer.*.js -247 bytes 127.07kB -0.19%

Files in assets/RawFileViewer.*.js:

  • ./src/shared/ContentsTable/TableEntries/BaseEntries/FileEntry.tsx → Total Size: 1.25kB

@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    #3772   +/-   ##
=======================================
  Coverage   98.72%   98.72%           
=======================================
  Files         824      824           
  Lines       14782    14782           
  Branches     4239     4249   +10     
=======================================
  Hits        14593    14593           
  Misses        182      182           
  Partials        7        7
Files with missing lines Coverage Δ
...ntentsTable/TableEntries/BaseEntries/FileEntry.tsx 100.00% <100.00%> (ø)
Components Coverage Δ
Assets 100.00% <ø> (ø)
Layouts 99.69% <ø> (ø)
Pages 98.29% <ø> (ø)
Services 99.36% <ø> (ø)
Shared 99.11% <100.00%> (ø)
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 40c29d5...5348c44. Read the comment docs.

@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 (40c29d5) to head (5348c44).
Report is 2 commits behind head on main.

✅ All tests successful. No failed tests found.

@@           Coverage Diff           @@
##             main    #3772   +/-   ##
=======================================
  Coverage   98.72%   98.72%           
=======================================
  Files         824      824           
  Lines       14782    14782           
  Branches     4247     4249    +2     
=======================================
  Hits        14593    14593           
  Misses        182      182           
  Partials        7        7
Files with missing lines Coverage Δ
...ntentsTable/TableEntries/BaseEntries/FileEntry.tsx 100.00% <100.00%> (ø)
Components Coverage Δ
Assets 100.00% <ø> (ø)
Layouts 99.69% <ø> (ø)
Pages 98.29% <ø> (ø)
Services 99.36% <ø> (ø)
Shared 99.11% <100.00%> (ø)
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 40c29d5...5348c44. 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 (40c29d5) to head (5348c44).
Report is 2 commits behind head on main.

✅ All tests successful. No failed tests found.

Additional details and impacted files 
@@           Coverage Diff           @@
##             main    #3772   +/-   ##
=======================================
  Coverage   98.72%   98.72%           
=======================================
  Files         824      824           
  Lines       14782    14782           
  Branches     4247     4249    +2     
=======================================
  Hits        14593    14593           
  Misses        182      182           
  Partials        7        7
Files with missing lines Coverage Δ
...ntentsTable/TableEntries/BaseEntries/FileEntry.tsx 100.00% <100.00%> (ø)
Components Coverage Δ
Assets 100.00% <ø> (ø)
Layouts 99.69% <ø> (ø)
Pages 98.29% <ø> (ø)
Services 99.36% <ø> (ø)
Shared 99.11% <100.00%> (ø)
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 40c29d5...5348c44. 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 (40c29d5) to head (5348c44).
Report is 2 commits behind head on main.

✅ All tests successful. No failed tests found.

@@           Coverage Diff           @@
##             main    #3772   +/-   ##
=======================================
  Coverage   98.72%   98.72%           
=======================================
  Files         824      824           
  Lines       14782    14782           
  Branches     4239     4249   +10     
=======================================
  Hits        14593    14593           
  Misses        182      182           
  Partials        7        7
Files with missing lines Coverage Δ
...ntentsTable/TableEntries/BaseEntries/FileEntry.tsx 100.00% <100.00%> (ø)
Components Coverage Δ
Assets 100.00% <ø> (ø)
Layouts 99.69% <ø> (ø)
Pages 98.29% <ø> (ø)
Services 99.36% <ø> (ø)
Shared 99.11% <100.00%> (ø)
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 40c29d5...5348c44. Read the comment docs.

@codecov
Copy link
Copy Markdown

codecov bot commented Feb 25, 2025 •
edited
Loading

Bundle Report

Changes will decrease total bundle size by 506 bytes (-0.0%) ⬇️. This is within the configured threshold ✅

Detailed changes
Bundle name Size Change
gazebo-production-system 6.13MB -259 bytes (-0.0%) ⬇️
gazebo-production-esm 6.2MB -247 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/RawFileViewer.*.js -247 bytes 127.07kB -0.19%

Files in assets/RawFileViewer.*.js:

  • ./src/shared/ContentsTable/TableEntries/BaseEntries/FileEntry.tsx → Total Size: 1.25kB
view changes for bundle: gazebo-production-system

Assets Changed:

Asset Name Size Change Total Size Change (%)
assets/RawFileViewer-legacy.*.js -259 bytes 127.3kB -0.2%

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

  • ./src/shared/ContentsTable/TableEntries/BaseEntries/FileEntry.tsx → Total Size: 1.25kB

@codecov-releaser
Copy link
Copy Markdown
Collaborator

codecov-releaser commented Feb 25, 2025 •
edited
Loading

✅ Deploy preview for gazebo ready!

Previews expire after 1 month automatically.

Storybook

Commit Created Cloud Enterprise
a05d183 Tue, 25 Feb 2025 14:51:22 GMT Expired Expired
b895c22 Tue, 25 Feb 2025 15:08:38 GMT Expired Expired
5348c44 Tue, 25 Feb 2025 17:02:43 GMT Cloud Enterprise

@spalmurray-codecov spalmurray-codecov marked this pull request as ready for review February 25, 2025 16:56
@spalmurray-codecov spalmurray-codecov added this pull request to the merge queue Feb 25, 2025
Merged via the queue into main with commit bc3d5dd Feb 25, 2025
62 checks passed
@spalmurray-codecov spalmurray-codecov deleted the spalmurray/fileentry-ts branch February 25, 2025 17:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

1 more reviewer

@nicholas-codecov nicholas-codecov nicholas-codecov approved these changes

Reviewers whose approvals may not affect merge requirements

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

@spalmurray-codecov @codecov-releaser @nicholas-codecov @spalmurray