Skip to content
This repository has been archived by the owner on Jun 3, 2021. It is now read-only.

* [WEEX-240] [android] feature update for weexsandbox #1088

Merged
merged 4 commits into from
Mar 27, 2018

Conversation

yuhun-alibaba
Copy link
Contributor

Weex SandBox

yuhun-alibaba and others added 4 commits March 27, 2018 15:55
     1. every page will has a runtime context independent of other page
     2. At the beginning of js, will use // {"framework" : "Rax"} or // {"framework" : "Vue"} to distinguish type
     3. if page type is neither Rax nor Vue, will use global context runtime
     4. delete some usless code and opt build tools, trim so
@weex-bot
Copy link

weex-bot commented Mar 27, 2018

Fails
🚫 Running your Dangerfile has Failed

Danger has errored

Error: TypeError

TypeError: Cannot read property 'code' of null
    at Object.<anonymous> (./dangerfile-android.js:98:126)
    at Runtime._execModule (/home/travis/build/apache/incubator-weex/node_modules/jest-runtime/build/index.js:446:13)
    at Runtime.requireModule (/home/travis/build/apache/incubator-weex/node_modules/jest-runtime/build/index.js:294:14)
    at /home/travis/build/apache/incubator-weex/node_modules/danger/distribution/runner/DangerfileRunner.js:136:33
    at ensureCleanDangerfile (/home/travis/build/apache/incubator-weex/node_modules/danger/distribution/runner/DangerfileRunner.js:176:5)
    at Object.<anonymous> (/home/travis/build/apache/incubator-weex/node_modules/danger/distribution/runner/DangerfileRunner.js:135:21)
    at step (/home/travis/build/apache/incubator-weex/node_modules/danger/distribution/runner/DangerfileRunner.js:32:23)
    at Object.next (/home/travis/build/apache/incubator-weex/node_modules/danger/distribution/runner/DangerfileRunner.js:13:53)
    at /home/travis/build/apache/incubator-weex/node_modules/danger/distribution/runner/DangerfileRunner.js:7:71
    at __awaiter (/home/travis/build/apache/incubator-weex/node_modules/danger/distribution/runner/DangerfileRunner.js:3:12)

Generated by 🚫 dangerJS

Copy link
Member

@misakuo misakuo left a comment

Choose a reason for hiding this comment

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

ok

@yuhun-alibaba yuhun-alibaba changed the title Apache master multicontext * [WEEX-240] [android] feature update for weexsandbox Mar 27, 2018
@asfgit asfgit merged commit 45ec92c into apache:master Mar 27, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants