Skip to content

Commit

Permalink
[devtools] Enable the framework upgrade test again
Browse files Browse the repository at this point in the history
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
  • Loading branch information
perryjrandall committed Jan 17, 2025
1 parent f7cc847 commit 2887619
Showing 1 changed file with 1 addition and 5 deletions.
6 changes: 1 addition & 5 deletions devtools/aptos-cargo-cli/src/lib.rs
Original file line number Diff line number Diff line change
Expand Up @@ -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!(
Expand Down

0 comments on commit 2887619

Please sign in to comment.