Skip to content

docs(apple): Add attachAllThreads option documentation#17258

Merged
itaybre merged 1 commit intomasterfrom
itay/docs-attach-all-threads
Apr 7, 2026
Merged

docs(apple): Add attachAllThreads option documentation#17258
itaybre merged 1 commit intomasterfrom
itay/docs-attach-all-threads

Conversation

@itaybre
Copy link
Copy Markdown
Contributor

@itaybre itaybre commented Apr 6, 2026

DESCRIBE YOUR PR

Document the new attachAllThreads option added in sentry-cocoa 9.9.0. This option enables attaching all threads with full stack traces to captured events.

Also documents the per-call attachAllThreads parameter override for SentrySDK.capture(event:), SentrySDK.capture(error:), SentrySDK.capture(exception:), and SentrySDK.capture(message:).

IS YOUR CHANGE URGENT?

  • Urgent deadline (GA date, etc.)
  • Other deadline
  • None: Not urgent, can wait up to 1 week+

PRE-MERGE CHECKLIST

  • Checked Vercel preview for correctness, including links
  • PR was reviewed and approved by any necessary SMEs (subject matter experts)
  • PR was reviewed and approved by a member of the Sentry docs team

Document the new attachAllThreads option added in sentry-cocoa 9.9.0,
including per-call override support for capture methods.
@vercel
Copy link
Copy Markdown

vercel bot commented Apr 6, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
sentry-docs Ready Ready Preview, Comment Apr 6, 2026 3:34pm
1 Skipped Deployment
Project Deployment Actions Updated (UTC)
develop-docs Ignored Ignored Apr 6, 2026 3:34pm

Request Review

Copy link
Copy Markdown
Contributor

@antonis antonis left a comment

Choose a reason for hiding this comment

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

LGTM

@itaybre itaybre merged commit f4970fe into master Apr 7, 2026
18 checks passed
@itaybre itaybre deleted the itay/docs-attach-all-threads branch April 7, 2026 21:11

When enabled, all threads are attached with full stack traces to all captured events. This requires briefly suspending all threads to collect their stack traces. When disabled (the default), only the current thread gets a stack trace.

This option requires `attachStacktrace` to also be enabled for it to have any effect.
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Suggested change
This option requires `attachStacktrace` to also be enabled for it to have any effect.
This option also requires `attachStacktrace` to be enabled for it to have any effect.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

I though it was already approved by the docs teams, I might have confused it with another PR.
Let me open a new one with this fix

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Ready: #17273

itaybre added a commit that referenced this pull request Apr 15, 2026
…ption documentation (#17273)

<!-- Use this checklist to make sure your PR is ready for merge. You may
delete any sections you don't need. -->

## DESCRIBE YOUR PR

This PR improves wording on Sentry-cocoa `attachAllThreads` option with
a comment from @coolguyzone on
#17258

## IS YOUR CHANGE URGENT?  

Help us prioritize incoming PRs by letting us know when the change needs
to go live.
- [ ] Urgent deadline (GA date, etc.): <!-- ENTER DATE HERE -->
- [ ] Other deadline: <!-- ENTER DATE HERE -->
- [x] None: Not urgent, can wait up to 1 week+

## SLA

- Teamwork makes the dream work, so please add a reviewer to your PRs.
- Please give the docs team up to 1 week to review your PR unless you've
added an urgent due date to it.
Thanks in advance for your help!

## PRE-MERGE CHECKLIST

*Make sure you've checked the following before merging your changes:*

- [ ] Checked Vercel preview for correctness, including links
- [ ] PR was reviewed and approved by any necessary SMEs (subject matter
experts)
- [ ] PR was reviewed and approved by a member of the [Sentry docs
team](https://github.com/orgs/getsentry/teams/docs)

## LEGAL BOILERPLATE

<!-- Sentry employees and contractors can delete or ignore this section.
-->

Look, I get it. The entity doing business as "Sentry" was incorporated
in the State of Delaware in 2015 as Functional Software, Inc. and is
gonna need some rights from me in order to utilize my contributions in
this here PR. So here's the deal: I retain all rights, title and
interest in and to my contributions, and by keeping this boilerplate
intact I confirm that Sentry can use, modify, copy, and redistribute my
contributions, under Sentry's choice of terms.

## EXTRA RESOURCES

- [Sentry Docs contributor guide](https://docs.sentry.io/contributing/)
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.

3 participants