Skip to content

[INTS25] - Updated various dependencies#51

Merged
jeessh merged 1 commit intomainfrom
jesse/upgrade-dependencies
Jun 21, 2025
Merged

[INTS25] - Updated various dependencies#51
jeessh merged 1 commit intomainfrom
jesse/upgrade-dependencies

Conversation

@jeessh
Copy link
Member

@jeessh jeessh commented Jun 7, 2025

Notion ticket link

Update node version

Implementation description

  • updated node, relevant packages, and Dockerfile
  • updated workflow cache

Steps to test

  1. run docker and ensure apis work

What should reviewers focus on?

Checklist

  • My PR name is descriptive and in imperative tense
  • My commit messages are descriptive and in imperative tense. My commits are atomic and trivial commits are squashed or fixup'd into non-trivial commits
  • I have run the appropriate linter(s)
  • I have requested a review from the PL, as well as other devs who have background knowledge on this PR or who will be building on top of this PR

@jeessh jeessh requested a review from mxc-maggiechen June 7, 2025 16:10
@jeessh jeessh self-assigned this Jun 7, 2025
@jeessh jeessh force-pushed the jesse/upgrade-dependencies branch 2 times, most recently from 98674d6 to 8b74b3f Compare June 7, 2025 17:09
ran linter

downgraded version to 20.12

update cache config
@jeessh jeessh force-pushed the jesse/upgrade-dependencies branch from 8843c0f to 528af13 Compare June 7, 2025 17:14
Copy link
Contributor

@mxc-maggiechen mxc-maggiechen 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

@jeessh jeessh changed the title feat: updated various dependencies [INTS25] - Updated various dependencies Jun 18, 2025
@jeessh jeessh merged commit b5a7742 into main Jun 21, 2025
1 check passed
@jeessh jeessh deleted the jesse/upgrade-dependencies branch June 21, 2025 15:49
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.

2 participants