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

Upgrade Typescript to v5 #7421

Draft
wants to merge 3 commits into
base: master
Choose a base branch
from
Draft

Upgrade Typescript to v5 #7421

wants to merge 3 commits into from

Conversation

ixrock
Copy link
Member

@ixrock ixrock commented Mar 28, 2023

No description provided.

@ixrock ixrock requested review from jansav and Iku-turso March 28, 2023 09:47
@ixrock ixrock requested a review from a team as a code owner March 28, 2023 09:47
@ixrock ixrock self-assigned this Mar 28, 2023
@ixrock ixrock marked this pull request as draft March 28, 2023 09:47
@github-actions
Copy link
Contributor

This pull request has conflicts, please resolve those before we can evaluate the pull request.

Signed-off-by: Roman <ixrock@gmail.com>
Signed-off-by: Roman <ixrock@gmail.com>
…t doesn't seem to work at all (catching the proper version by app/builder or whatsoever)

Signed-off-by: Roman <ixrock@gmail.com>
@ixrock ixrock assigned jansav and unassigned ixrock Mar 30, 2023
@ixrock
Copy link
Member Author

ixrock commented Mar 30, 2023

@jansav @Iku-turso i'm leaving this up to you since this is DX responsibility area and cause I'm stuck at this.

There are a lot of unknowns how to orchestrate all installation and running parallel processes while we use lerna for monorepos.
I've updated to tsc@5.x in infrastructure/typescript/package.json but nothing changed. Seems like it's not even considered in compilation for some reasons..

P.S. If you prefer to start from scratch (update TS) please close this PR.

@jansav jansav removed their assignment Apr 5, 2023
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.

None yet

2 participants