Skip to content

fix(demo-mode): make _sync_project_artifact_bundle failsafe #93376

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

Conversation

constantinius
Copy link
Contributor

Copy link

sentry-io bot commented Jun 11, 2025

🔍 Existing Issues For Review

Your pull request is modifying functions with the following pre-existing issues:

📄 File: src/sentry/demo_mode/tasks.py

Function Unhandled Issue
_sync_project_artifact_bundle ProjectArtifactBundle.DoesNotExist: ProjectArtifactBundle matching query does not exist. sen...
Event Count: 5

Did you find this useful? React with a 👍 or 👎

@github-actions github-actions bot added the Scope: Backend Automatically applied to PRs that change backend components label Jun 11, 2025
@constantinius constantinius enabled auto-merge (squash) June 11, 2025 21:13
@constantinius constantinius merged commit 8a54078 into master Jun 12, 2025
61 checks passed
@constantinius constantinius deleted the constantinius/fix/demo-mode/sync-project-artifact-bundle-failsafe branch June 12, 2025 07:10
Copy link

codecov bot commented Jun 12, 2025

Codecov Report

Attention: Patch coverage is 66.66667% with 1 line in your changes missing coverage. Please review.

✅ All tests successful. No failed tests found.

Files with missing lines Patch % Lines
src/sentry/demo_mode/tasks.py 66.66% 1 Missing ⚠️
Additional details and impacted files
@@             Coverage Diff             @@
##           master   #93376       +/-   ##
===========================================
+ Coverage   41.27%   80.76%   +39.49%     
===========================================
  Files       10259    10286       +27     
  Lines      592078   593033      +955     
  Branches    23028    23028               
===========================================
+ Hits       244401   478986   +234585     
+ Misses     347245   113615   -233630     
  Partials      432      432               

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
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