Skip to content

Conversation

@rosen-vladimirov
Copy link
Contributor

When there are no devices attached, CLI should start emulator and execute the action on it.
For example calling tns run android when there are no devices attached, the devicesService should start emulator and run the application in it.

Currently this is not working as the startEmulator method is starting the instance and then calls method for device detection.
However the checkForDevices method is async and it does not guarantee when the devices will be detected.
Instead use startLookingForDevices which is synchronous.

Fixes #1576

@rosen-vladimirov rosen-vladimirov self-assigned this Mar 9, 2016
@rosen-vladimirov rosen-vladimirov added this to the 1.7.0 milestone Mar 9, 2016
@teobugslayer
Copy link
Contributor

👍

When there are no devices attached, CLI should start emulator and execute the action on it.
For example calling `tns run android` when there are no devices attached, the devicesService should start emulator and run the application in it.

Currently this is not working as the startEmulator method is starting the instance and then calls method for device detection.
However the checkForDevices method is async and it does not guarantee when the devices will be detected.
Instead use startLookingForDevices which is synchronous.
@rosen-vladimirov rosen-vladimirov force-pushed the vladimirov/fix-run-after-start-emulator branch from dda6955 to cda83fe Compare March 9, 2016 14:32
@rosen-vladimirov
Copy link
Contributor Author

run ci

1 similar comment
@rosen-vladimirov
Copy link
Contributor Author

run ci

rosen-vladimirov added a commit that referenced this pull request Mar 9, 2016
…tart-emulator

Fix starting of emulator when no devices are attached
@rosen-vladimirov rosen-vladimirov merged commit 2ac0444 into release Mar 9, 2016
@rosen-vladimirov rosen-vladimirov deleted the vladimirov/fix-run-after-start-emulator branch March 9, 2016 15:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants