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

Are these libs used? #16548

Closed
wants to merge 2 commits into from
Closed

Are these libs used? #16548

wants to merge 2 commits into from

Conversation

mroz22
Copy link
Contributor

@mroz22 mroz22 commented Jan 22, 2025

... let me try.

also why (some of them) are they listed as connect dependencies? https://github.com/trezor/trezor-suite/blob/develop/scripts/list-outdated-dependencies/connect-dependencies.txt#L11

when deleted

image image

looks quite same to me 🤔 are these libs still needed? cc @tomasklim

@mroz22 mroz22 force-pushed the are-these-libs-used branch from dfb50d7 to c7fa917 Compare January 22, 2025 17:04
Copy link

github-actions bot commented Jan 22, 2025

🚀 Expo preview is ready!

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

Learn more about 𝝠 Expo Github Action

@mroz22 mroz22 added no-project This label is used to specify that PR doesn't need to be added to a project dependencies Pull requests that update a dependency file labels Jan 22, 2025
@mroz22 mroz22 force-pushed the are-these-libs-used branch from c7fa917 to a4ebc1c Compare January 23, 2025 08:44
@mroz22
Copy link
Contributor Author

mroz22 commented Jan 23, 2025

ok, there is a difference

image

image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dependencies Pull requests that update a dependency file no-project This label is used to specify that PR doesn't need to be added to a project
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant