Skip to content

ref(typing): type group_event_details#65309

Merged
JoshFerge merged 3 commits into
masterfrom
jferg/typing6
Feb 16, 2024
Merged

ref(typing): type group_event_details#65309
JoshFerge merged 3 commits into
masterfrom
jferg/typing6

Conversation

@JoshFerge

Copy link
Copy Markdown
Member

No description provided.

@JoshFerge JoshFerge requested a review from a team as a code owner February 16, 2024 04:56
@github-actions github-actions Bot added the Scope: Backend Automatically applied to PRs that change backend components label Feb 16, 2024

def issue_search_query_to_conditions(
query: str, group: Group, user: User, environments: Sequence[Environment]
query: str, group: Group, user: User | AnonymousUser, environments: Sequence[Environment]

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

in practice i don't think an AnonymousUser would ever make it in here, as to access the endpoint the user would have to be authed, although I don't think we have a way of defining the user type based on the type of request atm.

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

@codecov

codecov Bot commented Feb 16, 2024

Copy link
Copy Markdown

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (526b69c) 81.47% compared to head (ad5d0ef) 81.46%.
Report is 5 commits behind head on master.

❗ Current head ad5d0ef differs from pull request most recent head 99eb1e2. Consider uploading reports for the commit 99eb1e2 to get more accurate results

Additional details and impacted files
@@            Coverage Diff             @@
##           master   #65309      +/-   ##
==========================================
- Coverage   81.47%   81.46%   -0.01%     
==========================================
  Files        5255     5255              
  Lines      232112   232115       +3     
  Branches    45524    45524              
==========================================
- Hits       189106   189101       -5     
- Misses      37127    37132       +5     
- Partials     5879     5882       +3     
Files Coverage Δ
src/sentry/api/endpoints/group_event_details.py 83.87% <100.00%> (+0.17%) ⬆️
src/sentry/api/helpers/group_index/index.py 88.27% <100.00%> (+0.08%) ⬆️

... and 12 files with indirect coverage changes

@asottile-sentry asottile-sentry left a comment

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.

@JoshFerge JoshFerge enabled auto-merge (squash) February 16, 2024 15:32
@JoshFerge JoshFerge merged commit 2f87fbd into master Feb 16, 2024
@JoshFerge JoshFerge deleted the jferg/typing6 branch February 16, 2024 16:00
@github-actions github-actions Bot locked and limited conversation to collaborators Mar 3, 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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants