Skip to content

Commit c2001f9

Browse files
committed
publish: Inline validate_dependencies() fn
1 parent ba43404 commit c2001f9

File tree

1 file changed

+4
-10
lines changed

1 file changed

+4
-10
lines changed

src/controllers/krate/publish.rs

Lines changed: 4 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -284,8 +284,11 @@ pub async fn publish(app: AppState, req: BytesRequest) -> AppResult<Json<GoodCra
284284
VersionAction::Publish,
285285
)?;
286286

287+
for dep in &metadata.deps {
288+
validate_dependency(dep)?;
289+
}
290+
287291
// Link this new version to all dependencies
288-
validate_dependencies(&metadata.deps)?;
289292
add_dependencies(conn, &metadata.deps, version.id)?;
290293

291294
// Update all keywords for this crate
@@ -431,15 +434,6 @@ fn missing_metadata_error_message(missing: &[&str]) -> String {
431434
)
432435
}
433436

434-
#[instrument(skip_all)]
435-
pub fn validate_dependencies(deps: &[EncodableCrateDependency]) -> AppResult<()> {
436-
for dep in deps {
437-
validate_dependency(dep)?;
438-
}
439-
440-
Ok(())
441-
}
442-
443437
pub fn validate_dependency(dep: &EncodableCrateDependency) -> AppResult<()> {
444438
if !Crate::valid_name(&dep.name) {
445439
return Err(cargo_err(&format_args!(

0 commit comments

Comments
 (0)