Skip to content

Update package.json to v3.0.0#749

Merged
vakrilov merged 4 commits intomasterfrom
version3
Apr 13, 2017
Merged

Update package.json to v3.0.0#749
vakrilov merged 4 commits intomasterfrom
version3

Conversation

@vakrilov
Copy link
Copy Markdown
Contributor

package.json - dependencies and version updated.

CC @vchimev

@vakrilov vakrilov requested a review from sis0k0 April 13, 2017 11:54
Comment thread CHANGELOG.md Outdated
@@ -1,4 +1,76 @@
# 3.0.0 (2017-xx-xx)
- Compatible with Angular 4 and NativeScript 3.0

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

required TypeScript version 2.2 as breaking change?

Comment thread nativescript-angular/package.json Outdated
},
"nativescript": {}
"rxjs": "^5.0.1",
"reflect-metadata": "^0.1.8",
Copy link
Copy Markdown
Contributor

@vchimev vchimev Apr 13, 2017

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

dependency - "reflect-metadata": "~0.1.8"
or
devDependency - "reflect-metadata": "^0.1.8"

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No point to have them as a dev-deps. I'll move them

"reflect-metadata": "~0.1.8",
"punycode": "1.3.2",
"querystring": "0.2.0",
"url": "0.10.3"
Copy link
Copy Markdown
Contributor

@vchimev vchimev Apr 13, 2017

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Don't we need these anymore:

    "punycode": "1.3.2",
    "querystring": "0.2.0",
    "url": "0.10.3"

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yup

@vakrilov vakrilov merged commit 53f2bfc into master Apr 13, 2017
@vakrilov vakrilov deleted the version3 branch April 13, 2017 13:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants