Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

dev: Update TS from 4.9.5 -> ~5.7.3 #3647

Merged
merged 1 commit into from
Jan 10, 2025
Merged

Conversation

ajay-sentry
Copy link
Contributor

@ajay-sentry ajay-sentry commented Jan 10, 2025

Description

This PR bumps TS from ^4.9.5 -> ~5.7.3, effectively "pinning" TS to 5.7 so we can be particular about bumping up to 5.8 and later. This is because TS sometimes has breaking changes between minor versions.

For reference, TS 4.9.5 was released January 31st 2023, so we're getting around 2 years worth of version updates with this PR :)

Changelog

Whats new (up til 5.5... I used to like reading these "spark notes" but I guess they stopped doing this since 5.6) -> https://www.typescriptlang.org/docs/handbook/release-notes/typescript-5-0.html

Some of my favorite highlights from this TS update:

and a WHOLE BUNCH more

Screen.Recording.2025-01-10.at.10.29.49.AM.mov
Screenshot 2025-01-10 at 10 11 30 AM Screenshot 2025-01-10 at 10 13 23 AM

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.

const propsLink = pageConfig?.isExternalLink ? { href: path } : { to: path }
const propsLink = pageConfig?.isExternalLink
? { href: path }
: Component === 'a'
Copy link
Contributor Author

Choose a reason for hiding this comment

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

"a" link wasn't accounted for previously, now it is

@@ -23,7 +27,9 @@ function Breadcrumb({ paths = [], largeFont = false }) {
{defaultTo(to.children, to.text)}
</span>
) : (
<A to={to}>{defaultTo(to.children, to.text)}</A>
<A to={to} hook={undefined} isExternal={false}>
Copy link
Contributor Author

Choose a reason for hiding this comment

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

just set these to undefined to preserve original behavior

@ajay-sentry ajay-sentry changed the title dev: Update TS to latest version dev: Update TS fomr 4.9.5 -> ~5.7.3 Jan 10, 2025
@codecov-staging
Copy link

codecov-staging bot commented Jan 10, 2025

Bundle Report

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

Detailed changes
Bundle name Size Change
gazebo-staging-system 6.04MB 21 bytes (-0.0%) ⬇️
gazebo-staging-esm 6.09MB 13 bytes (-0.0%) ⬇️

Copy link

codecov bot commented Jan 10, 2025

Codecov Report

Attention: Patch coverage is 87.50000% with 1 line in your changes missing coverage. Please review.

Project coverage is 98.94%. Comparing base (bd76de4) to head (ee25c28).
Report is 2 commits behind head on main.

✅ All tests successful. No failed tests found.

Files with missing lines Patch % Lines
src/shared/AppLink/AppLink.tsx 83.33% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #3647      +/-   ##
==========================================
- Coverage   98.95%   98.94%   -0.01%     
==========================================
  Files         813      813              
  Lines       14599    14600       +1     
  Branches     4146     4148       +2     
==========================================
  Hits        14446    14446              
- Misses        146      147       +1     
  Partials        7        7              
Files with missing lines Coverage Δ
src/pages/RepoPage/context.tsx 100.00% <ø> (ø)
src/ui/Breadcrumb/Breadcrumb.tsx 100.00% <100.00%> (ø)
src/shared/AppLink/AppLink.tsx 96.96% <83.33%> (-3.04%) ⬇️
Components Coverage Δ
Assets 100.00% <ø> (ø)
Layouts 99.71% <ø> (ø)
Pages 98.66% <ø> (ø)
Services 99.34% <ø> (ø)
Shared 99.30% <83.33%> (-0.07%) ⬇️
UI 99.14% <100.00%> (-0.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 bd76de4...ee25c28. Read the comment docs.

@codecov-qa
Copy link

codecov-qa bot commented Jan 10, 2025

Codecov Report

Attention: Patch coverage is 87.50000% with 1 line in your changes missing coverage. Please review.

Project coverage is 98.94%. Comparing base (bd76de4) to head (ee25c28).
Report is 2 commits behind head on main.

✅ All tests successful. No failed tests found.

Files with missing lines Patch % Lines
src/shared/AppLink/AppLink.tsx 83.33% 1 Missing ⚠️
@@            Coverage Diff             @@
##             main    #3647      +/-   ##
==========================================
- Coverage   98.95%   98.94%   -0.01%     
==========================================
  Files         813      813              
  Lines       14599    14600       +1     
  Branches     4146     4148       +2     
==========================================
  Hits        14446    14446              
- Misses        146      147       +1     
  Partials        7        7              
Files with missing lines Coverage Δ
src/pages/RepoPage/context.tsx 100.00% <ø> (ø)
src/ui/Breadcrumb/Breadcrumb.tsx 100.00% <100.00%> (ø)
src/shared/AppLink/AppLink.tsx 96.96% <83.33%> (-3.04%) ⬇️
Components Coverage Δ
Assets 100.00% <ø> (ø)
Layouts 99.71% <ø> (ø)
Pages 98.66% <ø> (ø)
Services 99.34% <ø> (ø)
Shared 99.30% <83.33%> (-0.07%) ⬇️
UI 99.14% <100.00%> (-0.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 bd76de4...ee25c28. Read the comment docs.

@codecov-notifications
Copy link

codecov-notifications bot commented Jan 10, 2025

Codecov Report

Attention: Patch coverage is 87.50000% with 1 line in your changes missing coverage. Please review.

✅ All tests successful. No failed tests found.

Files with missing lines Patch % Lines
src/shared/AppLink/AppLink.tsx 83.33% 1 Missing ⚠️
@@            Coverage Diff             @@
##             main    #3647      +/-   ##
==========================================
- Coverage   98.95%   98.94%   -0.01%     
==========================================
  Files         813      813              
  Lines       14599    14600       +1     
  Branches     4139     4148       +9     
==========================================
  Hits        14446    14446              
- Misses        146      147       +1     
  Partials        7        7              
Files with missing lines Coverage Δ
src/pages/RepoPage/context.tsx 100.00% <ø> (ø)
src/ui/Breadcrumb/Breadcrumb.tsx 100.00% <100.00%> (ø)
src/shared/AppLink/AppLink.tsx 96.96% <83.33%> (-3.04%) ⬇️
Components Coverage Δ
Assets 100.00% <ø> (ø)
Layouts 99.71% <ø> (ø)
Pages 98.66% <ø> (ø)
Services 99.34% <ø> (ø)
Shared 99.30% <83.33%> (-0.07%) ⬇️
UI 99.14% <100.00%> (-0.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 bd76de4...ee25c28. Read the comment docs.

Copy link

codecov bot commented Jan 10, 2025

Bundle Report

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

Detailed changes
Bundle name Size Change
gazebo-production-system 6.04MB 21 bytes (-0.0%) ⬇️
gazebo-production-esm 6.09MB 13 bytes (-0.0%) ⬇️

@codecov-releaser
Copy link
Contributor

✅ Deploy preview for gazebo ready!

Previews expire after 1 month automatically.

Storybook

Commit Created Cloud Enterprise
ee25c28 Fri, 10 Jan 2025 18:29:08 GMT Cloud Enterprise

@ajay-sentry ajay-sentry added this pull request to the merge queue Jan 10, 2025
Merged via the queue into main with commit 5febc26 Jan 10, 2025
50 of 62 checks passed
@ajay-sentry ajay-sentry deleted the Ajay/ts-update-latest branch January 10, 2025 18:44
@ajay-sentry ajay-sentry changed the title dev: Update TS fomr 4.9.5 -> ~5.7.3 dev: Update TS from 4.9.5 -> ~5.7.3 Jan 21, 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.

3 participants