Skip to content

chore: merge release-patch in release #5130

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 5 commits into from
Nov 13, 2019
Merged

chore: merge release-patch in release #5130

merged 5 commits into from
Nov 13, 2019

Conversation

rosen-vladimirov
Copy link
Contributor

PR Checklist

What is the current behavior?

What is the new behavior?

Fixes/Implements/Closes #[Issue Number].

KristianDD and others added 5 commits October 23, 2019 11:58
…e-trace-log

chore: add trace for shouldMigrate caching
…same as the previous one

Currently CLI considers the hash that is the same as the previous as invalid and this way the application is restarted on device. As that hash is already handled, we should consider it as a valid hash.
Steps to reproduce :
```
`tns create ts_proj`  - choose ts tabs project

npm i tns-core-modules@rc --save --save-exact

npm i nativescript-dev-webpack@rc --save-dev --save-exact

node_modules/.bin/update-ns-webpack --deps –configs

tns run android

Change something in app.android.scss - the change is applied, no restart

Change something in app.ios.scss - application restarts
```
fix: handle correctly the situation when the compilation hash is the same as the previous one
@cla-bot cla-bot bot added the cla: yes label Nov 13, 2019
@Fatme Fatme merged commit 954189b into release Nov 13, 2019
@Fatme Fatme deleted the release-patch branch November 13, 2019 15:04
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