Skip to content

fix: make logger backwards compatible #4610

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 1 commit into from
May 13, 2019

Conversation

rosen-vladimirov
Copy link
Contributor

Several plugin use deleted logger methods in their hooks. Same is valid for nativescript-cloud extension.
To resolve this, get back the deleted methods, we'll delete them in 6.0.0 release.

Also, nativescript-cloud has an option --workflow, that is an object, while ours is boolean. This breaks its transpilation, so set ours to any.

PR Checklist

What is the current behavior?

What is the new behavior?

Fixes/Implements/Closes #[Issue Number].

Several plugin use deleted logger methods in their hooks. Same is valid for `nativescript-cloud` extension.
To resolve this, get back the deleted methods, we'll delete them in 6.0.0 release.

Also, `nativescript-cloud` has an option `--workflow`, that is an object, while ours is boolean. This breaks its transpilation, so set ours to `any`.
@rosen-vladimirov rosen-vladimirov added this to the 5.4.0 milestone May 10, 2019
@rosen-vladimirov rosen-vladimirov self-assigned this May 10, 2019
@ghost ghost added the new PR label May 10, 2019
@DimitarTachev DimitarTachev merged commit 201c2be into release May 13, 2019
@DimitarTachev DimitarTachev deleted the vladimirov/backwards-compatible branch May 13, 2019 06:35
@ghost ghost removed the new PR label May 13, 2019
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