Skip to content

Commit 987f647

Browse files
authored
Merge pull request #1160 from salesforcecli/t/2gp-readiness/w-21768097/block-conversion-not-in-project
@W-21768097 Block sf package convert calls outside of sfdx project directory
2 parents a9c21eb + e0a31ea commit 987f647

1 file changed

Lines changed: 4 additions & 9 deletions

File tree

src/commands/package/convert.ts

Lines changed: 4 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -15,7 +15,7 @@
1515
*/
1616

1717
import { Flags, loglevel, orgApiVersionFlagWithDeprecations, SfCommand } from '@salesforce/sf-plugins-core';
18-
import { Lifecycle, Messages, SfError, SfProject } from '@salesforce/core';
18+
import { Lifecycle, Messages, SfError } from '@salesforce/core';
1919
import {
2020
INSTALL_URL_BASE,
2121
Package,
@@ -37,6 +37,7 @@ export class PackageConvert extends SfCommand<PackageVersionCreateRequestResult>
3737
public static readonly examples = messages.getMessages('examples');
3838
public static readonly deprecateAliases = true;
3939
public static readonly aliases = ['force:package:convert'];
40+
public static readonly requiresProject = true;
4041
public static readonly flags = {
4142
loglevel,
4243
'target-dev-hub': requiredHubFlag,
@@ -128,13 +129,7 @@ export class PackageConvert extends SfCommand<PackageVersionCreateRequestResult>
128129
} else {
129130
this.spinner.start('Converting Package', 'Initializing');
130131
}
131-
// initialize the project instance if in a project
132-
let project: SfProject | undefined;
133-
try {
134-
project = await SfProject.resolve();
135-
} catch (err) {
136-
// ignore project is optional
137-
}
132+
138133
const result = await Package.convert(
139134
flags.package,
140135
flags['target-dev-hub'].getConnection(flags['api-version']),
@@ -148,7 +143,7 @@ export class PackageConvert extends SfCommand<PackageVersionCreateRequestResult>
148143
patchversion: flags['patch-version'] as string,
149144
codecoverage: flags['code-coverage'] as boolean,
150145
},
151-
project
146+
this.project
152147
);
153148

154149
switch (result.Status) {

0 commit comments

Comments
 (0)