Skip to content

Validate assemble_artifact arguments earlier#53333

Merged
Swatinem merged 6 commits into
masterfrom
swatinem/stricter-assemble
Jul 24, 2023
Merged

Validate assemble_artifact arguments earlier#53333
Swatinem merged 6 commits into
masterfrom
swatinem/stricter-assemble

Conversation

@Swatinem

Copy link
Copy Markdown
Contributor
  • Ensures that we always have a Project for ArtifactBundles
  • Ensures that there is always a release for a Releasebundle
  • Defaults the bundle_id earlier

@Swatinem Swatinem self-assigned this Jul 21, 2023
@Swatinem Swatinem requested a review from a team July 21, 2023 14:16
@github-actions github-actions Bot added the Scope: Backend Automatically applied to PRs that change backend components label Jul 21, 2023
Comment thread src/sentry/models/artifactbundle.py Outdated
- Ensures that we always have a `Project` for ArtifactBundles
- Ensures that there is always a release for a Releasebundle
- Defaults the `bundle_id` earlier
Comment thread src/sentry/tasks/assemble.py
Comment thread src/sentry/tasks/assemble.py Outdated
Comment thread src/sentry/tasks/assemble.py Outdated
Comment thread src/sentry/tasks/assemble.py
@codecov

codecov Bot commented Jul 24, 2023

Copy link
Copy Markdown

Codecov Report

Merging #53333 (e7c418a) into master (d16d615) will decrease coverage by 0.01%.
The diff coverage is 75.00%.

Additional details and impacted files
@@            Coverage Diff             @@
##           master   #53333      +/-   ##
==========================================
- Coverage   79.55%   79.54%   -0.01%     
==========================================
  Files        4942     4942              
  Lines      208639   208640       +1     
  Branches    35544    35546       +2     
==========================================
- Hits       165975   165972       -3     
- Misses      37605    37607       +2     
- Partials     5059     5061       +2     
Impacted Files Coverage Δ
src/sentry/tasks/assemble.py 86.89% <69.23%> (-0.54%) ⬇️
src/sentry/models/artifactbundle.py 96.86% <100.00%> (-0.02%) ⬇️

... and 3 files with indirect coverage changes

Swatinem and others added 2 commits July 24, 2023 10:23
Co-authored-by: Riccardo Busetti <riccardo.busetti@sentry.io>
@Swatinem Swatinem merged commit fb82ab0 into master Jul 24, 2023
@Swatinem Swatinem deleted the swatinem/stricter-assemble branch July 24, 2023 10:11
armenzg pushed a commit that referenced this pull request Jul 24, 2023
- Ensures that we always have a `Project` for ArtifactBundles
- Ensures that there is always a release for a Releasebundle
- Defaults the `bundle_id` earlier

Co-authored-by: Riccardo Busetti <riccardo.busetti@sentry.io>
@github-actions github-actions Bot locked and limited conversation to collaborators Aug 8, 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.

3 participants