[TIMOB-10599] Tooling: Update JSCA generator to include deprecation information
GitHub Issue | n/a |
---|---|
Type | New Feature |
Priority | Medium |
Status | Closed |
Resolution | Fixed |
Resolution Date | 2012-10-17T23:25:16.000+0000 |
Affected Version/s | Release 3.0.0 |
Fix Version/s | Release 3.0.0, Release 3.1.0, 2012 Sprint 21 Core, 2012 Sprint 21 |
Components | Tooling |
Labels | core |
Reporter | Arthur Evans |
Assignee | Bill Dawson |
Created | 2012-08-24T11:12:05.000+0000 |
Updated | 2012-12-21T05:18:04.000+0000 |
Description
This ticket relies on the linked Ti Studio ticket. Once the new JSCA format has been defined, we need to support it.
Per the linked issues, in the short term, we're going to place the deprecation warning (if any) into the JSCA description. We should probably append it after the description, like:
\{summary}. Deprecated since {version}. {deprecation note}
I've tried several variations on this, and it appears that Studio ignores most formatting (except the ), so Adding it as a separate paragraph isn't useful. We will also need to add the deprecation flag for members, which Studio does not support yet. Need to find out whether this change will require a rev of the JSCA version #.Testing notes
Build and release the test branch. Be sure _not_ to build it with
scons build_jsca=0
since obviously we need the JSCA to be built. :)Get the latest nightly of Titanium Studio.
Open Titanium Studio.
Create a Titanium Project and target the version of the mobile sdk you are testing. That should be either 3.1.0 or 3.0.0 (the latter would be when you are testing the backport Pull Request).
Open app.js.
Begin typing
var x = Ti.UI.Android.LINKIFY_ALL;
. Once you type pastTi.UI.Android.
, be sure to pay attention to the code completion's description ofLINKIFY_ALL
. It should include deprecation information in it.Finish typing that line and hit Enter. After a few seconds the yellow warning indicator should show on
LINKIFY_ALL
and if you hover over it you should see the warning that it is deprecated.Master PR: https://github.com/appcelerator/titanium_mobile/pull/3185 3_0_X PR: https://github.com/appcelerator/titanium_mobile/pull/3250