Skip to content

Conversation

@Turbo87
Copy link
Member

@Turbo87 Turbo87 commented Oct 28, 2024

This PR makes the error handling for the "version XXX already exists" case a little easier on the 👀

@Turbo87 Turbo87 added C-internal 🔧 Category: Nonessential work that would make the codebase more consistent or clear A-backend ⚙️ labels Oct 28, 2024
@Turbo87 Turbo87 enabled auto-merge October 28, 2024 13:06
@Turbo87 Turbo87 merged commit 3947c3b into rust-lang:main Oct 28, 2024
8 checks passed
@Turbo87 Turbo87 deleted the simplify-duplicate-version-error branch October 28, 2024 13:17
@codecov
Copy link

codecov bot commented Oct 28, 2024

Codecov Report

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

Project coverage is 88.80%. Comparing base (3ce70e7) to head (7f0b098).
Report is 3 commits behind head on main.

Files with missing lines Patch % Lines
src/controllers/krate/publish.rs 87.50% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #9774      +/-   ##
==========================================
- Coverage   88.80%   88.80%   -0.01%     
==========================================
  Files         289      289              
  Lines       29809    29808       -1     
==========================================
- Hits        26472    26471       -1     
  Misses       3337     3337              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

A-backend ⚙️ C-internal 🔧 Category: Nonessential work that would make the codebase more consistent or clear

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant