Skip to content
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

fix: ensure PYTEST_DEBUG_TEMPROOT exists #54725

Merged
merged 2 commits into from
Aug 15, 2023

Conversation

joshuarli
Copy link
Member

@lobsterkatie was running into this

@github-actions github-actions bot added the Scope: Backend Automatically applied to PRs that change backend components label Aug 14, 2023
@codecov
Copy link

codecov bot commented Aug 14, 2023

Codecov Report

Merging #54725 (cdc5ced) into master (14935b6) will decrease coverage by 0.01%.
Report is 2 commits behind head on master.
The diff coverage is 0.00%.

Additional details and impacted files
@@            Coverage Diff             @@
##           master   #54725      +/-   ##
==========================================
- Coverage   79.80%   79.80%   -0.01%     
==========================================
  Files        5000     5000              
  Lines      212384   212388       +4     
  Branches    36188    36188              
==========================================
+ Hits       169487   169488       +1     
- Misses      37684    37688       +4     
+ Partials     5213     5212       -1     
Files Changed Coverage Δ
src/sentry/testutils/pytest/sentry.py 75.63% <0.00%> (-1.57%) ⬇️

... and 6 files with indirect coverage changes

@joshuarli joshuarli merged commit 6f2c9ec into master Aug 15, 2023
56 of 57 checks passed
@joshuarli joshuarli deleted the fix-mkdir-pytest_debug_temproot branch August 15, 2023 20:04
@github-actions github-actions bot locked and limited conversation to collaborators Aug 31, 2023
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