Skip to content
This repository has been archived by the owner on Jun 7, 2023. It is now read-only.

Commit

Permalink
Remove use-native-renames flag for ImportIt
Browse files Browse the repository at this point in the history
Summary: This flag is always set now, so we don't need it (less things to translate to CodeSync)

Reviewed By: jailby

Differential Revision: D45283010

fbshipit-source-id: 1bcda581d20fad4ea79218fc1b281a1f09b1029d
  • Loading branch information
bigfootjon authored and facebook-github-bot committed Apr 26, 2023
1 parent e5a574b commit aa02e3b
Showing 1 changed file with 1 addition and 13 deletions.
14 changes: 1 addition & 13 deletions src/importit/phase/ImportItSyncPhase.php
Original file line number Diff line number Diff line change
Expand Up @@ -29,7 +29,6 @@ final class ImportItSyncPhase extends \Facebook\ShipIt\ShipItPhase {
private bool $applyToLatest = false;
private ?string $applyToTarget = null;
private bool $shouldDoSubmodules = true;
private bool $nativeRenames = false;

public function __construct(
private (function(ShipItChangeset): ShipItChangeset) $filter,
Expand Down Expand Up @@ -116,15 +115,6 @@ final public function getCLIArguments(
return $this->shouldDoSubmodules;
},
),
shape(
'long_name' => 'native-renames',
'description' =>
'Use native renames instead of deleting and adding files',
'write' => $_ ==> {
$this->nativeRenames = true;
return $this->nativeRenames;
},
),
];
}

Expand Down Expand Up @@ -167,9 +157,7 @@ final public function getCLIArguments(
$manifest->getSourcePath(),
);
await $source_repo->genSetBranch($manifest->getSourceBranch());
if ($this->nativeRenames) {
$source_repo->setUseNativeRenames(true);
}
$source_repo->setUseNativeRenames(true);
return await $source_repo->genChangesetAndBaseRevisionForPullRequest(
$pr_number,
$expected_head_rev,
Expand Down

0 comments on commit aa02e3b

Please sign in to comment.