Skip to content

chore: Fix typos - #8241

Merged
sentrivana merged 1 commit into
masterfrom
fix/typos
Nov 11, 2026
Merged

sentrivana merged 1 commit into
masterfrom
fix/typos

Conversation

@sentry-junior

@sentry-junior sentry-junior Bot commented Nov 11, 2026

Copy link
Copy Markdown
Contributor

Fixes clear-cut misspellings across comments, docstrings, script output, and a few local or test-only identifiers. Found with codespell and typos, and every hit was checked by hand.

Nothing public changes. The renamed identifiers are a local variable in sentry_sdk/utils.py (should_supress_context), a local in scripts/find_raise_from_none.py, a test-local variable, and three test function names.

These were left alone on purpose:

  • dependant: FastAPI's own API name
  • checkin: Sentry crons terminology
  • deprecat: regex prefix in .github/release.yml
  • need_code_loation: old method name quoted in MIGRATION_GUIDE.md
  • baGGage: tests that header matching ignores case
  • Chunked test strings like "hel" and "ons"
  • Typos in CHANGELOG.md, since those are historical release notes

via @sentrivana.

--

View Session [Sentry]

Co-Authored-By: Ivana Kellyer <131587164+sentrivana@users.noreply.github.com>
@sentrivana
sentrivana marked this pull request as ready for review November 11, 2026 10:00
@sentrivana
sentrivana requested a review from a team as a code owner November 11, 2026 10:00
@sentrivana
sentrivana enabled auto-merge (squash) November 11, 2026 10:01
@sentrivana
sentrivana merged commit 7c90371 into master Nov 11, 2026
170 checks passed
@sentrivana
sentrivana deleted the fix/typos branch November 11, 2026 10:09
@github-actions

Copy link
Copy Markdown
Contributor

Coverage Results 📊

✅ 138990 passed | ⏭️ 7669 skipped | Total: 146659 | Pass Rate: 94.77% | Execution Time: 445m 50s

📊 Comparison with Base Branch

Metric Change
Total Tests 📈 +1544
Passed Tests 📈 +1546
Failed Tests —
Skipped Tests 📉 -2

All tests are passing successfully.

✅ Patch coverage is 100.00% (2 of 2 changed executable lines covered; target 80%).
Project statement coverage is 90.25% (unchanged from base (305a20d) to head (c431214)).

Changed files with executable lines (1)
File Patch coverage Changed executable lines
sentry_sdk/utils.py 100.00% 2/2 covered
Coverage diff
@@            Coverage Diff             @@
##        master     #8241       +/-##
==========================================
  Coverage    90.25%    90.25%        —%
==========================================
  Files          203       203         —
  Tracked lines     27114     27114         —
  Branches     10136     10136         —
==========================================
  Hits         24470     24470         —
  Misses        2644      2644         —
  Partials      1538      1538         —

Generated by Coverage Action

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.

1 participant