Skip to content

Commit e060cb3

Browse files
committed
chore: resolve review comments
1 parent 459d3f6 commit e060cb3

File tree

3 files changed

+2
-15
lines changed

3 files changed

+2
-15
lines changed

src/commands/force/org/clone.ts

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -121,10 +121,10 @@ export class OrgCloneCommand extends SfCommand<SandboxProcessObject> {
121121
});
122122

123123
const { sandboxReq, srcSandboxName } = await createSandboxRequest(true, flags.definitionfile, logger, varargs);
124-
125124
logger.debug('Calling clone with SandboxRequest: %s and SandboxName: %s ', sandboxReq, srcSandboxName);
126125
flags['target-org'].getConnection(flags['api-version']);
127-
return flags['target-org'].cloneSandbox(sandboxReq, srcSandboxName, { wait: flags.wait });
126+
// eslint-disable-next-line @typescript-eslint/no-non-null-assertion
127+
return flags['target-org'].cloneSandbox(sandboxReq, srcSandboxName!, { wait: flags.wait });
128128
} else {
129129
throw new SfError(
130130
messages.getMessage('commandOrganizationTypeNotSupport', [OrgTypes.Sandbox]),

src/commands/org/list.ts

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -51,7 +51,6 @@ export class OrgListCommand extends SfCommand<OrgListResult> {
5151

5252
private flags!: Interfaces.InferredFlags<typeof OrgListCommand.flags>;
5353

54-
// eslint-disable-next-line sf-plugin/should-parse-flags
5554
public async run(): Promise<OrgListResult> {
5655
const [{ flags }, fileNames] = await Promise.all([this.parse(OrgListCommand), getAuthFileNames()]);
5756
this.flags = flags;

src/shared/sandboxRequest.ts

Lines changed: 0 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -20,18 +20,6 @@ export const generateSboxName = async (): Promise<string> => {
2020
return generated;
2121
};
2222

23-
export async function createSandboxRequest(
24-
isClone: true | boolean,
25-
definitionFile: string | undefined,
26-
logger?: Logger | undefined,
27-
properties?: Record<string, string | undefined>
28-
): Promise<{ sandboxReq: SandboxRequest; srcSandboxName: string }>;
29-
export async function createSandboxRequest(
30-
isClone: false,
31-
definitionFile: string | undefined,
32-
logger?: Logger | undefined,
33-
properties?: Record<string, string | undefined>
34-
): Promise<{ sandboxReq: SandboxRequest }>;
3523
export async function createSandboxRequest(
3624
isClone = false,
3725
definitionFile: string | undefined,

0 commit comments

Comments
 (0)