From f6b53eb4ab226e435a626e6b0c1da0618681017a Mon Sep 17 00:00:00 2001 From: Perry Randall Date: Thu, 16 Jan 2025 13:19:12 -1000 Subject: [PATCH] [devtools] Enable the framework upgrade test again Previously we disabled this because it was broken on PR Now that it is fixed on PR enable again to make sure it doesnt break! Test Plan: add label, should be triggered on PR --- devtools/aptos-cargo-cli/src/lib.rs | 6 +----- 1 file changed, 1 insertion(+), 5 deletions(-) diff --git a/devtools/aptos-cargo-cli/src/lib.rs b/devtools/aptos-cargo-cli/src/lib.rs index 0ec428e0a8483..27854ddb91232 100644 --- a/devtools/aptos-cargo-cli/src/lib.rs +++ b/devtools/aptos-cargo-cli/src/lib.rs @@ -272,17 +272,13 @@ impl AptosCargoCommand { // Determine if any relevant files or packages were changed #[allow(unused_assignments)] - let mut relevant_changes_detected = detect_relevant_changes( + let relevant_changes_detected = detect_relevant_changes( RELEVANT_FILE_PATHS_FOR_FRAMEWORK_UPGRADE_TESTS.to_vec(), RELEVANT_PACKAGES_FOR_FRAMEWORK_UPGRADE_TESTS.to_vec(), changed_files, affected_package_paths, ); - // TODO: remove this! This is a temporary fix to disable - // the framework upgrade test while we debug the failures. - relevant_changes_detected = false; - // Output if relevant changes were detected that require the framework upgrade // test. This will be consumed by Github Actions and used to run the test. println!(