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

Update Android profiling instructions #4362

Merged
merged 6 commits into from
Nov 22, 2024
Merged

Conversation

gaearon
Copy link
Contributor

@gaearon gaearon commented Nov 20, 2024

I couldn't get any of this to work last time I tried. I assumed the entire page is completely broken and outdated. Turns out, all of it is still relevant, it's just recommending tools that haven't worked for many years. This changes the recommendation to tools that actually work.

I couldn't get any of this to work last time I tried. I assumed the entire page is completely broken and outdated. Turns out, all of it is still relevant, it's just recommending tools that haven't worked for many years. This changes the recommendation to tools that actually work.
Copy link

netlify bot commented Nov 20, 2024

Deploy Preview for react-native ready!

Name Link
🔨 Latest commit 10662f0
🔍 Latest deploy log https://app.netlify.com/sites/react-native/deploys/67406d0cf2227400080f1d6d
😎 Deploy Preview https://deploy-preview-4362--react-native.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@javache
Copy link
Member

javache commented Nov 20, 2024

Thanks for this! We're looking to migrate our in-depth tracing markers to be based on perfetto (we've switched to this workflow at Meta) and will look to make this part of the open-source tooling too.

docs/profiling.md Outdated Show resolved Hide resolved
@gaearon
Copy link
Contributor Author

gaearon commented Nov 20, 2024

Addressed + added info about method profiling

@blakef
Copy link
Contributor

blakef commented Nov 21, 2024

There's no reason that build should be failing, just tried it locally and it built fine...

@cipolleschi
Copy link
Contributor

Linter is failing with:
Screenshot 2024-11-22 at 10 58 03

@blakef
Copy link
Contributor

blakef commented Nov 22, 2024

Ok, so the strange unrelated infra issue seems to have resolved itself after the last commit.

@gaearon
Copy link
Contributor Author

gaearon commented Nov 22, 2024

lmao "it's easy to get confused" is not the meaning of "easy" the linter was designed to prevent :D

@javache javache merged commit aaac547 into facebook:main Nov 22, 2024
4 checks passed
@gaearon gaearon deleted the patch-9 branch November 22, 2024 15:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants