You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@weex.apache.org by Hanks10100 <gi...@git.apache.org> on 2017/08/14 07:16:14 UTC
[GitHub] incubator-weex pull request #596: [jsfm] Refactor the file structure of weex...
GitHub user Hanks10100 opened a pull request:
https://github.com/apache/incubator-weex/pull/596
[jsfm] Refactor the file structure of weex runtime
+ [x] Add type check in the method of `TaskCenter`.
+ [x] Add directive filter for Element attributes.
+ [x] Convert `Element`, `Document`, `Node`, `Comment` into ES6 class.
+ [x] Use `WeexElement` instead of `element-types` to create new type of Weex component.
+ [x] Fix the wrong build path in `build_from_source.sh` to make the CI pass.
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/Hanks10100/incubator-weex jsfm-feature-0.22
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/incubator-weex/pull/596.patch
To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:
This closes #596
----
commit 6ecd723f053b1885a2afaaea3ad918fff71807ce
Author: Hanks <zh...@gmail.com>
Date: 2017-07-29T16:12:07Z
* [jsfm] update vanilla framework
commit 2ecf8565f8da844caea3069f551aba6c5a79a54b
Author: Hanks <zh...@gmail.com>
Date: 2017-08-03T03:28:42Z
+ [jsfm] add directive filter in runtime
commit d8bcbf242693c93757322533ec1612b03c680a18
Author: Hanks <zh...@gmail.com>
Date: 2017-08-03T03:45:27Z
* [jsfm] filter directives in attrs
commit 6fed87d488e5eaa7718b0cda5ddead6aa09a8238
Author: Hanks <zh...@gmail.com>
Date: 2017-08-03T03:47:36Z
Merge branch 'jsfm-feature-buffer' into jsfm-feature-beta
commit 66804f2eae0ba411f68e5086d8621eb91e293419
Author: Hanks <zh...@gmail.com>
Date: 2017-08-03T03:50:48Z
* [build] build weex-js-framework and weex-js-runtime
commit 25b0101dff7635932718d8d5feee2ad9e9a2f1dc
Author: Hanks <zh...@gmail.com>
Date: 2017-08-07T07:27:33Z
* [jsfm] use ES6 class in vdom
commit 0729ac590a7e69aa279de973273fa228507f77f1
Author: Hanks <zh...@gmail.com>
Date: 2017-08-07T07:49:36Z
* [jsfm] refactor the file structure in runtime
commit 2f7921ece78bdf60ff419b7c0973b359599f16cb
Author: Hanks <zh...@gmail.com>
Date: 2017-08-07T07:58:39Z
* [jsfm] modify the file path in test
commit e9ad371a3ce8eb5c7c4175b9d4fe117b2bdfecdc
Author: Hanks <zh...@gmail.com>
Date: 2017-08-07T08:37:30Z
* [jsfm] refactor element-types
commit 9510e8b896cac47fd84016b43ff171c82a1ee072
Author: Hanks <zh...@gmail.com>
Date: 2017-08-08T03:08:36Z
Merge branch 'jsfm-feature-vanilla' into jsfm-feature-es6
commit f3ebd620b93d2af1528018cd3567e201d3c90a6f
Author: Hanks <zh...@gmail.com>
Date: 2017-08-08T03:31:13Z
+ [build] add build script for vanilla framework
commit 479d770121f39be80e49e50974fd04e3c1e2a0a7
Author: Hanks <zh...@gmail.com>
Date: 2017-08-08T03:47:58Z
- [jsfm] remove useless apis in vanilla framework
commit 41c787747aeedae52a08e761172b280bc814f67d
Author: Hanks <zh...@gmail.com>
Date: 2017-08-08T04:11:38Z
* [jsfm] publish js runtime / legacy / vanilla frameworks
commit 229740a583b540e01e50fccfc3273bfc9415921f
Author: Hanks <zh...@gmail.com>
Date: 2017-08-08T09:01:41Z
Merge branch '0.16-dev' into jsfm-feature-0.22
commit 8294acfc2c5a5268edf1bd048981a578e5a9c253
Author: Hanks <zh...@gmail.com>
Date: 2017-08-08T09:11:20Z
* [jsfm] change the file name case in vdom
commit bade95514eb57eaf0dec7a4bf42f07b33f9c9aa6
Author: Hanks <zh...@gmail.com>
Date: 2017-08-08T16:26:17Z
* [jsfm] refactor the logic in runtime init
commit 9e13c86051e4e8e615ea0be0ae86e7237b546494
Author: Hanks <zh...@gmail.com>
Date: 2017-08-10T08:27:05Z
* [test] refactor the unit test cases of runtime
commit 500cfe8fd151c5a7cf0abeaa0552c55b65d14cbf
Author: Hanks <zh...@gmail.com>
Date: 2017-08-10T12:02:39Z
+ [test] add test cases for render real instance
commit 65c1af51e7ce144119220ca04ee719e99059baf6
Author: Hanks <zh...@gmail.com>
Date: 2017-08-10T12:06:15Z
* [jsfm] add type check in the method of TaskCenter
commit 8d6c0d2803c83866398832763d935a5e412c76ba
Author: Hanks <zh...@gmail.com>
Date: 2017-08-14T03:44:37Z
Merge branch '0.16-dev' into jsfm-feature-0.22
commit c8e9cdb6c2fd6c904b0fa99ef6d6be5a4f0b1e3e
Author: Hanks <zh...@gmail.com>
Date: 2017-08-14T06:04:01Z
* [build] update build scripts to copy the right path
commit e9f9df58ad899aecd339614d97c465d392ac097a
Author: Hanks <zh...@gmail.com>
Date: 2017-08-14T06:11:16Z
* [build] build weex-js-framework@0.21.9
----
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---
[GitHub] incubator-weex pull request #596: [jsfm] Refactor the file structure of weex...
Posted by asfgit <gi...@git.apache.org>.
Github user asfgit closed the pull request at:
https://github.com/apache/incubator-weex/pull/596
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---
[GitHub] incubator-weex issue #596: [jsfm] Refactor the file structure of weex runtim...
Posted by weex-bot <gi...@git.apache.org>.
Github user weex-bot commented on the issue:
https://github.com/apache/incubator-weex/pull/596
<!--
12 failure: Code file html5/r..., Code file package..., Code file package..., Code file package..., Code file html5/r..., Code file html5/r..., Code file html5/t..., Code file html5/t..., Code file html5/t..., Code file html5/t..., Code file package..., Code file package...
3 warning: No Changelog chan..., These new JS file..., This PR should up...
1 messages
-->
<table>
<thead>
<tr>
<th width="50"></th>
<th width="100%" data-danger-table="true">Fails</th>
</tr>
</thead>
<tbody><tr>
<td>:no_entry_sign:</td>
<td>Code file html5/runtime/bridge/normalize.js does not have the copyright header.</td>
</tr>
<tr>
<td>:no_entry_sign:</td>
<td>Code file packages/weex-js-framework/index.min.js does not have the copyright header.</td>
</tr>
<tr>
<td>:no_entry_sign:</td>
<td>Code file packages/weex-js-runtime/index.min.js does not have the copyright header.</td>
</tr>
<tr>
<td>:no_entry_sign:</td>
<td>Code file packages/weex-legacy-framework/index.min.js does not have the copyright header.</td>
</tr>
<tr>
<td>:no_entry_sign:</td>
<td>Code file html5/runtime/utils.js does not have the copyright header.</td>
</tr>
<tr>
<td>:no_entry_sign:</td>
<td>Code file html5/runtime/vdom/directive.js does not have the copyright header.</td>
</tr>
<tr>
<td>:no_entry_sign:</td>
<td>Code file html5/test/unit/runtime/runner/helper.js does not have the copyright header.</td>
</tr>
<tr>
<td>:no_entry_sign:</td>
<td>Code file html5/test/unit/runtime/runner/vanilla.js does not have the copyright header.</td>
</tr>
<tr>
<td>:no_entry_sign:</td>
<td>Code file html5/test/unit/runtime/runner/vue.js does not have the copyright header.</td>
</tr>
<tr>
<td>:no_entry_sign:</td>
<td>Code file html5/test/unit/runtime/vdom/directive.js does not have the copyright header.</td>
</tr>
<tr>
<td>:no_entry_sign:</td>
<td>Code file packages/weex-html5/index.min.js does not have the copyright header.</td>
</tr>
<tr>
<td>:no_entry_sign:</td>
<td>Code file packages/weex-vanilla-framework/index.min.js does not have the copyright header.</td>
</tr>
</tbody>
</table>
<table>
<thead>
<tr>
<th width="50"></th>
<th width="100%" data-danger-table="true">Warnings</th>
</tr>
</thead>
<tbody><tr>
<td>:warning:</td>
<td>No Changelog changes!</td>
</tr>
<tr>
<td>:warning:</td>
<td>These new JS files do not have Flow enabled: html5/runtime/utils.js, html5/runtime/vdom/directive.js, html5/test/unit/runtime/runner/helper.js, html5/test/unit/runtime/runner/vanilla.js, html5/test/unit/runtime/runner/vue.js, html5/test/unit/runtime/vdom/directive.js, packages/weex-html5/index.js, packages/weex-html5/index.min.js, packages/weex-vanilla-framework/index.js, packages/weex-vanilla-framework/index.min.js</td>
</tr>
<tr>
<td>:warning:</td>
<td>This PR should update related documents as well. </td>
</tr>
</tbody>
</table>
<table>
<thead>
<tr>
<th width="50"></th>
<th width="100%" data-danger-table="true">Messages</th>
</tr>
</thead>
<tbody><tr>
<td>:book:</td>
<td>According to the blame info, we recommended @luics , @Jinjiang to be the reviewers.</td>
</tr>
</tbody>
</table>
<p align="right">
Generated by :no_entry_sign: <a href="http://github.com/danger/danger-js/">dangerJS</a>
</p>
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---