Function cargo::ops::write_manifest_upgrades

pub fn write_manifest_upgrades(
    ws: &Workspace<'_>,
    upgrades: &HashMap<(String, SourceId), Version>,
    dry_run: bool,
) -> CargoResult<bool>

Update manifests with upgraded versions, and write to disk. Based on cargo-edit. Returns true if any file has changed.

Some of the checks here are duplicating checks already done in upgrade_manifests/upgrade_dependency. Why? Let’s say upgrade_dependency has found that dependency foo was eligible for an upgrade. But foo can occur in multiple manifest files, and even multiple times in the same manifest file, and may be pinned, renamed, etc. in some of the instances. So we still need to check here which dependencies to actually modify. So why not drop the upgrade map and redo all checks here? Because then we’d have to query the registries again to find the latest versions.