Skip to content

Commit

Permalink
move-build: Remove duplicate function
Browse files Browse the repository at this point in the history
## Description

`CompiledPackage::published_dependency_ids` is a duplicate of
`CompiledPackage::get_dependency_storage_package_ids`. Get rid of it and
replace its only call-site.

## Test plan

CI
  • Loading branch information
amnn committed Feb 13, 2025
1 parent 2c1d1d9 commit 28bbe98
Show file tree
Hide file tree
Showing 2 changed files with 1 addition and 5 deletions.
2 changes: 1 addition & 1 deletion crates/sui-core/src/unit_tests/auth_unit_test_utils.rs
Original file line number Diff line number Diff line change
Expand Up @@ -130,7 +130,7 @@ pub async fn upgrade_package_on_single_authority(
gas_payment,
package_id,
modules,
package.published_dependency_ids(),
package.get_dependency_storage_package_ids(),
(upgrade_cap, Owner::AddressOwner(sender)),
UpgradePolicy::COMPATIBLE,
digest,
Expand Down
4 changes: 0 additions & 4 deletions crates/sui-move-build/src/lib.rs
Original file line number Diff line number Diff line change
Expand Up @@ -607,10 +607,6 @@ impl CompiledPackage {
error: error_message.join("\n"),
})
}

pub fn published_dependency_ids(&self) -> Vec<ObjectID> {
self.dependency_ids.published.values().cloned().collect()
}
}

impl Default for BuildConfig {
Expand Down

0 comments on commit 28bbe98

Please sign in to comment.