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

Major dependency update #55

Merged
merged 15 commits into from
Apr 25, 2024
Merged

Major dependency update #55

merged 15 commits into from
Apr 25, 2024

Conversation

SplittyDev
Copy link
Member

@SplittyDev SplittyDev commented Apr 23, 2024

Abstract

This PR updates lots of dependencies. Some of them require code changes (mostly the next, axum and metrics upgrades), which have been implemented and tested.

Major Changes

Web

  • Upgrade node (19/20 -> 21)
    • Upgrade node in server dockerfile (20 -> 21)
    • Upgrade node in web dockerfile (19 -> 21)
    • Add .nvmrc for node 21
    • Add node 21 to CI matrix
    • Drop node [16, 18] from CI matrix (not supported anymore)
  • Upgrade NextJS (13 -> 14)
    • Fix SSR issue with DeleteButton component
    • Fix hydration issues in privacy page

Server

  • Upgrade axum (0.6 -> 0.7)
    • Fix server compile errors caused by major axum API changes
    • Introduce axum-extra dependency (axum dropped the headers feature)
    • Upgrade tower-http (0.4 -> 0.5) as required by axum 0.7
  • Upgrade metrics (0.21 -> 0.22)
    • Fix compile issues resulting from metrics API changes in server code

Minor Changes

  • ci: Update node matrix to include 21 and exclude unsupported versions ([16, 18])
  • web: Update lockfile with lots of semver-compatible dependency updates (no breakage observed)
  • server-workspace: Format all files (cargo +nightly fmt)
  • server-workspace: Fix minor clippy lints
  • server-workspace: Update async-trait (0.1.68 -> 0.1.80)
  • server-shared: Update once_cell (1.18 -> 1.19)
  • server-db: Update diesel (lockfile)
  • server-db: Update deadpool (0.9.5 -> 0.11) and deadpool-diesel (0.4.1 -> 0.6)
  • server: Update regex (1.9 -> 1.10)
  • server: Update tracing (0.1.37 -> 0.1.40)
  • server: Update metrics-exporter-prometheus (0.12 -> 0.14)

Other Changes

  • deny: Update config to version 2
    • This allows us to drop many properties that now have deny as their default value
  • deny: Add missing git source for bincache
  • web: Add BlueOak-1.0.0 to allowed licenses
    • A single package uses that one, and it seems to be some snowflake license that's basically just MIT. We can safely use it, but it's super annoying that people use these weird-ass licenses from time to time..

@SplittyDev SplittyDev force-pushed the task/update-deps-23042024 branch from 944aaa9 to 5165130 Compare April 23, 2024 08:42
@Neotamandua Neotamandua self-requested a review April 23, 2024 08:55
Copy link
Member

@Neotamandua Neotamandua left a comment

Choose a reason for hiding this comment

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

LGTM

Co-authored-by: Neotamandua <107320179+Neotamandua@users.noreply.github.com>
@SplittyDev SplittyDev merged commit e067d1f into development Apr 25, 2024
6 checks passed
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