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

Chore: bump foundation deps #16258

Merged
merged 7 commits into from
Jan 9, 2025
Merged

Chore: bump foundation deps #16258

merged 7 commits into from
Jan 9, 2025

Conversation

Lemonexe
Copy link
Contributor

@Lemonexe Lemonexe commented Jan 8, 2025

Description

Update most Foundation-related dependencies.

major version: none

minor version:

  • @eslint/js
  • eslint
  • eslint-plugin-jest
  • eslint-plugin-react-hooks
  • typescript-eslint
  • react-hook-form
  • @hookform/resolvers
  • systeminformation
  • int64-buffer

patch version:

  • eslint-plugin-react
  • express
  • idb
  • uuid

not updated:

  • electron keeping v32, more info in #15766
  • tiny-secp256k1 TODO in #12261
  • electron-store blocked by #14482

ℹ️ For reference, last bump foundation deps PR was #15752
👁️ I skimmed through all code changes except eslint-related packages; found nothing suspicious ✅

Related Issue

Resolve #16262

QA

👁️ Besides CI checks, I have tested locally:

  • suite dev web
    • resetting app storage (test idb persistence)
  • suite dev desktop
    • resetting app storage (test idb persistence)
    • react-hook-form & @hookform/resolvers tested in SignVerify page
    • systeminformation output in debug logs
    • superifical testing of google auth server: responds to GET localhost:3005/status
  • suite desktop linux build & windows build
    • app builds & runs
  • local suite native android
    • react-hook-form & @hookform/resolvers tested on Send form

Copy link

github-actions bot commented Jan 8, 2025

🚀 Expo preview is ready!

  • Project → trezor-suite-preview
  • Platforms → android, ios
  • Scheme → trezorsuitelite
  • Runtime Version → 21
  • More info

Learn more about 𝝠 Expo Github Action

@Lemonexe Lemonexe force-pushed the chore/bump-foundation-deps branch from 3f81b6b to 6b3d11f Compare January 8, 2025 13:50
@Lemonexe Lemonexe force-pushed the chore/bump-foundation-deps branch from 6b3d11f to b00deec Compare January 8, 2025 16:05
@Lemonexe
Copy link
Contributor Author

Lemonexe commented Jan 8, 2025

This test is known to be broken since #14553, authors were notified.

e2e-test-suite-web (@group_wallet, trezor-user-env-unix bitcoin-regtest, 1)
add-account-types.test.ts

@Lemonexe Lemonexe marked this pull request as ready for review January 8, 2025 16:42
@Lemonexe Lemonexe requested review from szymonlesisz, matejkriz and a team as code owners January 8, 2025 16:42
@Lemonexe Lemonexe merged commit 00cb7c4 into develop Jan 9, 2025
78 of 79 checks passed
@Lemonexe Lemonexe deleted the chore/bump-foundation-deps branch January 9, 2025 15:48
@STew790
Copy link
Contributor

STew790 commented Jan 15, 2025

QA OK
Send form working OK.
Naming of accoutns also OK.
Importing of accounts on mobile also OK

Info
25.1.2 995410a

@STew790 STew790 added the QA OK Issue passed QA without any blocker label Jan 15, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
QA OK Issue passed QA without any blocker
Projects
Status: ✅ Approved
Development

Successfully merging this pull request may close these issues.

Bump foundation deps (2025.01)
3 participants