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
chore(all): update template, update tests #653
Conversation
m1ga
commented
Jan 18, 2024
- remove all mobileweb, tizen, windows, blackberry parts
- move "ti.appcelerator.org" to "ti.tide.io" (it's overwriting the tisdk templates again!)
- fix tests
Stupid Emulator tests :-) But now it's working again |
This is a major breaking change. Can you bump the version in the |
package.json
Outdated
@@ -6,7 +6,7 @@ | |||
"titanium", | |||
"mobile" | |||
], | |||
"version": "5.1.9", | |||
"version": "5.2.0", |
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.
Let's do a major release.
"version": "5.2.0", | |
"version": "6.0.0", |
While you're updating the package.json
, could you please set the "author": "TiDev, Inc. <npm@tidev.io>"
and remove the maintainers
.
We really should bump the min Node.js version to Node 16. Even then, we should drop Node 16 and 18. Ugh, maybe a larger effort. Dunno.
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.
sure, done 👍 I think if we do a major update we should merge #652 too so the Arm Mac users are happy again 😄
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.
Ok, I'll merge this and wait to publish until #652 has been reviewed/merged.