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

[email protected], fix peer deps, upgrades and fixes #27086

Merged
merged 8 commits into from
Apr 30, 2024
Merged

Conversation

arelra
Copy link
Member

@arelra arelra commented Apr 29, 2024

What does this change?

Update TypeScript to v5.3.3 to align with @guardian/commercial.

Fixing all peer dependencies became a thread pulling exercise of also updating: @guardian packages, eslint, babel, react, jest etc and fixing ensuing TS, babel and jest issues.

There's probably more upgrades we could do but this PR just fixes all peer deps and resulting errors from the TS upgrade.

The client side code in Frontend is mostly deprecated, it only runs on pages that Frontend still renders and the commercial codebase still runs on (primarily tag pages, video, audio and galleries atm).

Checklist

  • Tested locally, and on CODE if necessary

@arelra arelra requested review from a team as code owners April 29, 2024 17:28
Copy link
Contributor

@mxdvl mxdvl left a comment

Choose a reason for hiding this comment

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

As tidy as it gets

@arelra arelra merged commit 1d913b1 into main Apr 30, 2024
4 checks passed
@arelra arelra deleted the ravi/ts-5.3.3 branch April 30, 2024 09:10
@prout-bot
Copy link
Collaborator

Seen on ADMIN-PROD (merged by @arelra 11 minutes and 53 seconds ago)

@prout-bot
Copy link
Collaborator

Seen on FRONTS-PROD (merged by @arelra 12 minutes ago)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants