[ALOY-1367] Guide on how to submit a PR to the Alloy repo
|Reporter||Feon Sua Xin Miao|
DescriptionWe don't have a concise developer guide for Alloy. It would be great if there is a page that provides step by step guide for anyone who wants to contribute to Alloy. Some of the basic requirement for a PR to warrant a review: 1. Sign the CLA 2. Link to JIRA ticket 3. Include test app 4. All tests should pass 5. Code Conventions, this would be solved by integrate a linter.
- Fokke Zandbergen 2016-03-29
Some scripts that really we should wrap in simple NodeJS script:
*How to create a test app*
node tools/create_test.js TICKET*How to run a single test app*
jake app:run dir=testing/TICKET*How to generate known good code for a test app*
node tools/create_generated_code.js testing/TICKET*How to test a single test app compile*
app=testing/TICKET jake test:spec\[compile.js\]*How to test a single test app plus Alloy*
app=testing/TICKET jake test:all*How to test all*
- Chee Kiat Ng 2016-04-21 [~fmiao] i think this should go to TIDOCS? [~bimmel] fyi.
- Fokke Zandbergen 2016-05-02 [~cng] It sure should, but it needs some more work/thoughts and ideally we'd improve the scripts to simplify. But then, since [~cbarber] is moving Alloy into Titanium this all might be old news soon.
- Chris Barber 2016-05-02 I wouldn't put any energy into this. Instead I would clean up the TiSDK contributor guide, of which there is a ticket for somewhere.
- Chee Kiat Ng 2016-05-02