Skip to content

Merge release into master #4488

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

Merged
merged 11 commits into from
Apr 1, 2019
Merged

Conversation

DimitarTachev
Copy link
Contributor

No description provided.

rosen-vladimirov and others added 11 commits March 28, 2019 16:02
The emulator detection interval is not required for CLI and in fact it just uses CPU and memory. Also, it causes some issues when you do not hava JAVA installed.
So remove start of emulator detection interval and allow it to be started from the API, when CLI is used as a library.
In case JAVA_HOME is not set and there's no java executable in PATH, calls to `avdmanager` fail with error. However, on macOS this also leads to system prompt to install JAVA.
To handle this, do not execute `avdmanager` in case there's no JAVA_HOME or java in PATH.
Add clearInterval in timers API, so we can safely remove the cached timers from our internal cache. Without this, we end up calling many clearIntervals for timers that have already been cleared.
…ns-fixes

Device/emulator detection fixes
chore: update TypeScript dev dependencies
@cla-bot cla-bot bot added the cla: yes label Apr 1, 2019
@ghost ghost assigned DimitarTachev Apr 1, 2019
@ghost ghost added the new PR label Apr 1, 2019
@DimitarTachev DimitarTachev merged commit 2761adc into master Apr 1, 2019
@ghost ghost removed the new PR label Apr 1, 2019
@DimitarTachev DimitarTachev deleted the tachev/merge-release-into-master branch April 1, 2019 10:39
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.

2 participants