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

[packages] unify TSConfig package version #19671

Merged
merged 6 commits into from Oct 24, 2022
Merged

Conversation

Simek
Copy link
Collaborator

@Simek Simek commented Oct 24, 2022

Why

Refs #18204
Refs #19670

How

This PR aims to unify the @tsconfig/node package used version across the workspace.

One more dependency which uses @tsconfig/node12 is @expo/rudder-sdk-node, but I have opted-out updating this one, since I'm not sure what's in the future for this package. Let me know if I also should bump this one.

Test Plan

Modified packages have been build correctly, also tests seems to do not yield any errors.

Checklist

@expo-bot expo-bot added bot: suggestions ExpoBot has some suggestions bot: passed checks ExpoBot has nothing to complain about and removed bot: suggestions ExpoBot has some suggestions labels Oct 24, 2022
Copy link
Contributor

@Kudo Kudo left a comment

Choose a reason for hiding this comment

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

LGTM, thanks for this improvement. not sure whether it's related, but sdk packages ci is broken. should we rebuild the js files?

@Simek Simek force-pushed the @simek/packages-use-tsconfig14 branch from ad63c3b to cfbdc15 Compare October 24, 2022 14:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bot: passed checks ExpoBot has nothing to complain about
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants