Skip to content

Fix typos #68659

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

Merged
merged 1 commit into from
Apr 10, 2024
Merged

Fix typos #68659

merged 1 commit into from
Apr 10, 2024

Conversation

ViToni
Copy link
Contributor

@ViToni ViToni commented Apr 10, 2024

Fix usage of cound.

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.

@ViToni ViToni requested review from a team as code owners April 10, 2024 21:41
@github-actions github-actions bot added Scope: Frontend Automatically applied to PRs that change frontend components Scope: Backend Automatically applied to PRs that change backend components labels Apr 10, 2024
Copy link
Contributor

🚨 Warning: This pull request contains Frontend and Backend changes!

It's discouraged to make changes to Sentry's Frontend and Backend in a single pull request. The Frontend and Backend are not atomically deployed. If the changes are interdependent of each other, they must be separated into two pull requests and be made forward or backwards compatible, such that the Backend or Frontend can be safely deployed independently.

Have questions? Please ask in the #discuss-dev-infra channel.

@ryan953 ryan953 added the Trigger: getsentry tests Once code is reviewed: apply label to PR to trigger getsentry tests label Apr 10, 2024
Copy link

codecov bot commented Apr 10, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 79.74%. Comparing base (9381168) to head (95113dc).
Report is 3 commits behind head on master.

Additional details and impacted files
@@             Coverage Diff             @@
##           master   #68659       +/-   ##
===========================================
+ Coverage   54.00%   79.74%   +25.74%     
===========================================
  Files        5423     6412      +989     
  Lines      227784   283988    +56204     
  Branches    39188    48927     +9739     
===========================================
+ Hits       123023   226479   +103456     
+ Misses     104375    57122    -47253     
- Partials      386      387        +1     
Files Coverage Δ
src/sentry/issues/escalating_issues_alg.py 100.00% <ø> (+53.06%) ⬆️
...app/utils/replays/hooks/useLogReplayDataLoaded.tsx 0.00% <ø> (ø)

... and 2224 files with indirect coverage changes

@ryan953 ryan953 merged commit 6dffe62 into getsentry:master Apr 10, 2024
53 of 55 checks passed
c298lee pushed a commit that referenced this pull request Apr 12, 2024
Fix usage of `cound`.

### 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.
@github-actions github-actions bot locked and limited conversation to collaborators Apr 26, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Scope: Backend Automatically applied to PRs that change backend components Scope: Frontend Automatically applied to PRs that change frontend components Trigger: getsentry tests Once code is reviewed: apply label to PR to trigger getsentry tests
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants