-
-
Notifications
You must be signed in to change notification settings - Fork 338
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
feat(expo): Dynamically resolve sentry-cli path in expo source maps upload script #3507
Conversation
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for the PR!
Android (legacy) Performance metrics 🚀
|
Android (new) Performance metrics 🚀
|
This is now giving me:
Using on windows |
@rklomp same issue here unfortunately. |
I am now using |
Could you give an example of how you use that? |
I used this command in powershell. Just google how to set environment variable in windows. You can also set |
Thanks! |
📢 Type of change
📜 Description
The hardcoded path could fail for certain mono repo scenarios and also for our own sample application.
💚 How did you test it?
sample app
📝 Checklist
sendDefaultPII
is enabled🔮 Next steps