Skip to content

docs(ruby): Document attempt_threshold for sentry-sidekiq - #19329

Merged
sl0thentr0py merged 1 commit into
getsentry:masterfrom
hypermkt:docs-ruby-sidekiq-attempt-threshold
Sep 14, 2026
Merged

sl0thentr0py merged 1 commit into
getsentry:masterfrom
hypermkt:docs-ruby-sidekiq-attempt-threshold

Conversation

@hypermkt

@hypermkt hypermkt commented Sep 10, 2026 •

Copy link
Copy Markdown
Contributor

DESCRIBE YOUR PR

attempt_threshold is only mentioned in the changelog, so there's no reference for it in the docs. Someone asked about it on the PR that added it, and the maintainer said it should be documented:

good point, we should definitely document this and related features as it's kinda tricky 😅 I'll make sure we get it done!

— getsentry/sentry-ruby#2503 (comment)

IS YOUR CHANGE URGENT?

  • Urgent deadline (GA date, etc.): YYYY-MM-DD
  • Other deadline: YYYY-MM-DD
  • No deadline: Not urgent, can wait up to 1 week+

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

LEGAL BOILERPLATE

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.

This option lets a job skip error reporting until a given attempt, but it
was only mentioned in the changelog. The maintainer noted it should be
documented in getsentry/sentry-ruby#2503 (comment).

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
@vercel

vercel Bot commented Sep 10, 2026

Copy link
Copy Markdown

@hypermkt is attempting to deploy a commit to the Sentry Team on Vercel.

A member of the Team first needs to authorize it.

@github-actions github-actions Bot added the Priority: Normal Docs review has no urgent deadline label Sep 10, 2026
@hypermkt
hypermkt marked this pull request as ready for review September 10, 2026 05:21
@codeowner-assignment
codeowner-assignment Bot requested a review from a team September 10, 2026 05:22
@sl0thentr0py
sl0thentr0py enabled auto-merge (squash) September 14, 2026 18:26
@vercel

vercel Bot commented Sep 14, 2026 •

Copy link
Copy Markdown

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

Project Deployment Actions Updated
sentry-docs Ready Ready Preview Sep 14, 2026 8:03pm UTC
1 Skipped Deployment
Project Deployment Actions Updated
develop-docs Ignored Ignored Preview Sep 14, 2026 8:03pm UTC

Request Review

@sl0thentr0py
sl0thentr0py merged commit 3f4c620 into getsentry:master Sep 14, 2026
29 checks passed
@hypermkt
hypermkt deleted the docs-ruby-sidekiq-attempt-threshold branch September 17, 2026 14:00
@hypermkt

Copy link
Copy Markdown
Contributor Author

Thanks for merging!

inventarSarah pushed a commit that referenced this pull request Sep 29, 2026
## DESCRIBE YOUR PR

`attempt_threshold` is only mentioned in the changelog, so there's no
reference for it in the docs. Someone asked about it on the PR that
added it, and the maintainer said it should be documented:

> good point, we should definitely document this and related features as
it's kinda tricky 😅 I'll make sure we get it done!
>
> — [getsentry/sentry-ruby#2503
(comment)](getsentry/sentry-ruby#2503 (comment))

## IS YOUR CHANGE URGENT?

- [ ] Urgent deadline (GA date, etc.): YYYY-MM-DD
- [ ] Other deadline: YYYY-MM-DD
- [x] No deadline: Not urgent, can wait up to 1 week+

## 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

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.

Co-authored-by: Claude Opus 5 (1M context) <noreply@anthropic.com>

This branch was successfully deployed

1 active deployment
Preview – sentry-docs — b464f8b4 Deployed Sep 14, 2026 by vercel[bot]
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Priority: Normal Docs review has no urgent deadline

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants