Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

feat: add app hang / anr #187

Merged
merged 9 commits into from
Feb 22, 2024
Merged

feat: add app hang / anr #187

merged 9 commits into from
Feb 22, 2024

Conversation

buenaflor
Copy link
Contributor

@buenaflor buenaflor commented Feb 20, 2024

📜 Description

Adds options to configure app hang / anr

💡 Motivation and Context

Closes #179

💚 How did you test it?

📝 Checklist

You have to check all boxes before merging:

  • I reviewed the submitted code.
  • I added tests to verify the changes.
  • No new PII added or SDK only sends newly added PII if sendDefaultPII is enabled.
  • I updated the docs if needed.
  • Review from the native team if needed.
  • No breaking change or entry added to the changelog.

🔮 Next steps

Copy link
Contributor

github-actions bot commented Feb 20, 2024

Messages
📖 Do not forget to update Sentry-docs with your feature once the pull request gets approved.

Generated by 🚫 dangerJS against 56ad029

Copy link

codecov bot commented Feb 20, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (618a235) 77.46% compared to head (56ad029) 77.77%.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #187      +/-   ##
==========================================
+ Coverage   77.46%   77.77%   +0.31%     
==========================================
  Files          27       27              
  Lines         426      432       +6     
  Branches       56       56              
==========================================
+ Hits          330      336       +6     
  Misses         70       70              
  Partials       26       26              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@buenaflor buenaflor marked this pull request as ready for review February 20, 2024 02:31
@buenaflor buenaflor requested a review from romtsn as a code owner February 20, 2024 02:31
CHANGELOG.md Show resolved Hide resolved
Copy link
Member

@romtsn romtsn left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@buenaflor buenaflor merged commit 41c921a into main Feb 22, 2024
10 checks passed
@buenaflor buenaflor deleted the feat/app-hang branch February 22, 2024 12:31
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

The Ability to Adjust App Hang Preferences
3 participants