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.

Sign up for GitHub

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

Jump to bottom

chore(deps): Update all deps and tsconfig #204

Merged
merged 1 commit into from
Feb 13, 2024
Merged

chore(deps): Update all deps and tsconfig #204

merged 1 commit into from
Feb 13, 2024

Conversation

jimmycallin
Copy link
Contributor

@jimmycallin jimmycallin commented Feb 13, 2024 •
edited
Loading

  • I have added automatic tests where applicable
  • The PR title is suitable as a release note
  • The PR contains a description of what has been changed
  • The description contains manual test instructions

Changes

  • Updates all deps to latest
  • Use same tsconfig as in ftrack-web-widget with some modern settings turned on ("module": "NodeNext", "verbatimModuleSyntax": true).

I did a diff of the build before and after, and the build is identical except for the type keyword being added to the imports (and something in the sourcemaps)

image image

Test

  • Tests should pass

@jimmycallin jimmycallin requested a review from a team as a code owner February 13, 2024 15:24
@jimmycallin jimmycallin merged commit 99d6d72 into main Feb 13, 2024
6 checks passed
@jimmycallin jimmycallin deleted the deps branch February 13, 2024 15:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@gismya gismya gismya approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@jimmycallin @gismya