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
[TIMOB-24794] Updated to ioslib@1.4.3 which updates to node-ios-devic… #9130
Conversation
…e@1.3.3 and adds support for Node.js 8.
node-ios-device supporting Node 8 does not mean that Titanium supports Node 8 right? How is the status of supporting node 6/7 right now? I think I lost track about it. |
@hansemannn I have been using Node 8 for a while now with Titanium and it seems to work just fine. Even |
@hansemannn @ewieberappc What can we do to move this PR on? I'd like to get this in and merged so that I can update #9048 with it and then merge that in (moving from checked-in node_modules to using npm5's package-lock.json). |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Talked to Chris about some lines - LGTM, but would like to request a CR from @janvennemann just to be sure.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM! 👍
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
FR Passed. Able to build for sim and device using recent Xcodes (except 8.3.3 because of a known bug). No issues encountered and built apps ran without error.
[TIMOB-24794] Updated to ioslib@1.4.3 which updates to node-ios-device@1.3.3 and adds support for Node.js 8.
JIRA: https://jira.appcelerator.org/browse/TIMOB-24794