Skip to content

fix(api): test symbol sources in DetailedProjectSerializer #56473

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 1 commit into from
Sep 19, 2023
Merged

fix(api): test symbol sources in DetailedProjectSerializer #56473

merged 1 commit into from
Sep 19, 2023

Conversation

cathteng
Copy link
Member

@cathteng cathteng commented Sep 19, 2023 •
edited
Loading

#56414 (comment)

can you also test sentry:symbol_sources, which was the key our team was specifically interested in.

@codecov
Copy link

codecov bot commented Sep 19, 2023

Codecov Report

Merging #56473 (6a26623) into master (dc18d59) will increase coverage by 0.00%.
Report is 3 commits behind head on master.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master   #56473   +/-   ##
=======================================
  Coverage   78.64%   78.65%           
=======================================
  Files        5078     5078           
  Lines      218272   218269    -3     
  Branches    36952    36952           
=======================================
+ Hits       171670   171683   +13     
+ Misses      41064    41045   -19     
- Partials     5538     5541    +3

@cathteng cathteng marked this pull request as ready for review September 19, 2023 15:47
@cathteng cathteng requested review from Swatinem and a team September 19, 2023 15:47
@cathteng cathteng merged commit c141d1f into master Sep 19, 2023
@cathteng cathteng deleted the cathy/project-options/test-symbol-sources branch September 19, 2023 17:01
@github-actions github-actions bot locked and limited conversation to collaborators Oct 5, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@JoshFerge JoshFerge JoshFerge approved these changes

@Swatinem Swatinem Awaiting requested review from Swatinem

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@cathteng @JoshFerge