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

fix: missing devDependency #15220

Merged
merged 1 commit into from
Nov 5, 2024
Merged

fix: missing devDependency #15220

merged 1 commit into from
Nov 5, 2024

Conversation

peter-sanderson
Copy link
Contributor

@peter-sanderson peter-sanderson commented Nov 5, 2024

import * as messages from '@trezor/protobuf/messages.json';

is used in that package

image

Copy link

github-actions bot commented Nov 5, 2024

🚀 Expo preview is ready!

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

Learn more about 𝝠 Expo Github Action

@mroz22
Copy link
Contributor

mroz22 commented Nov 5, 2024

how comes it passed the checks?

@peter-sanderson
Copy link
Contributor Author

@mroz22 I haven investigated, but in branch with ESLint9 it triggered the check.

@peter-sanderson peter-sanderson enabled auto-merge (rebase) November 5, 2024 12:39
@peter-sanderson peter-sanderson merged commit 0828717 into develop Nov 5, 2024
32 checks passed
@peter-sanderson peter-sanderson deleted the fix-missing-dep branch November 5, 2024 12:44
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