-
-
Notifications
You must be signed in to change notification settings - Fork 195
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
move device discovery to debug command as it is done for run command #3008
Changes from 1 commit
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -15,13 +15,17 @@ export class LiveSyncCommandHelper implements ILiveSyncCommandHelper { | |
return availablePlatforms; | ||
} | ||
|
||
/** | ||
* Method sets up configuration, before calling livesync and expects that devices are already discovered. | ||
* @param devices List of discovered devices | ||
* @param liveSyncService Service expected to do the actual livesyncing | ||
* @param platform The platform for which the livesync will be ran | ||
*/ | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. The JSDocs are more applicable for the interface ( Also you have to define the return type of the method:
|
||
public async executeLiveSyncOperation(devices: Mobile.IDevice[], liveSyncService: ILiveSyncService, platform: string): Promise<void> { | ||
if (!devices || !devices.length) { | ||
this.$errors.failWithoutHelp("Unable to find applicable devices to execute operation and unable to start emulator when platform is not specified."); | ||
} | ||
|
||
await this.$devicesService.detectCurrentlyAttachedDevices({ shouldReturnImmediateResult: false, platform }); | ||
|
||
const workingWithiOSDevices = !platform || this.$mobileHelper.isiOSPlatform(platform); | ||
const shouldKeepProcessAlive = this.$options.watch || !this.$options.justlaunch; | ||
if (workingWithiOSDevices && shouldKeepProcessAlive) { | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
you have to define the type of the parameter:
https://github.com/Microsoft/TypeScript/wiki/JsDoc-support-in-JavaScript