Skip to content

Conversation

dmacvicar
Copy link
Contributor

@dmacvicar dmacvicar commented Oct 1, 2025

  • Use latest node LTS and Go version
  • Bump Go dependencies

Not addressed:

  • npm dependencies
  • Perhaps @types/node needs to be upgraded too given the base node dependency

Smoke tested with Docker Desktop 4.47.0

@dmacvicar dmacvicar marked this pull request as ready for review October 1, 2025 08:34
@dmacvicar dmacvicar requested a review from Pive01 October 1, 2025 08:34
Copy link
Collaborator

@Pive01 Pive01 left a comment

Choose a reason for hiding this comment

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

Seems like it still builds and works correctly so LGTM 👍
Thanks for addressing the issues

@dmacvicar dmacvicar merged commit 3ca192a into main Oct 1, 2025
@alexrashed alexrashed deleted the refresh-base-deps branch October 1, 2025 08:56
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