weex-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From weex-bot <...@git.apache.org>
Subject [GitHub] incubator-weex issue #1147: [Android] Merge WeexCore-master to master once.
Date Fri, 04 May 2018 08:38:04 GMT
Github user weex-bot commented on the issue:

    https://github.com/apache/incubator-weex/pull/1147
  
    
    <!--
      2 failure:  Code file weex_co..., Code file weex_co...
      4 warning:  No Changelog chan..., These new JS file..., Potential BREAK C..., Potential
BREAK C...
      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 weex_core/Source/IPC/IPCMessageJS.h does not have the copyright
header.</td>
        </tr>
      
    <tr>
          <td>:no_entry_sign:</td>
          <td>Code file weex_core/Source/android/jniprebuild/jniheader/WXJsFunctions_jni.h
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: pre-build/weex-rax-extra-api.js</td>
        </tr>
      
    <tr>
          <td>:warning:</td>
          <td>Potential BREAK CHANGE. Modify public in android/sdk/src/main/java/com/taobao/weex/bridge/NativeInvokeHelper.java
without metion it in commit message. You'd better add '@notdanger' in your commit log. </td>
        </tr>
      
    <tr>
          <td>:warning:</td>
          <td>Potential BREAK CHANGE. Modify public in android/sdk/src/main/java/com/taobao/weex/layout/measurefunc/TextContentBoxMeasurement.java
without metion it in commit message. You'd better add '@notdanger' in your commit log. </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>danger test finished.</td>
        </tr>
      </tbody>
    </table>
    
    
    <p align="right">
      Generated by :no_entry_sign: <a href="http://github.com/danger/danger-js/">dangerJS</a>
    </p>



---

Mime
View raw message