You are viewing a plain text version of this content. The canonical link for it is here.

Posted to notifications@apisix.apache.org by GitBox <gi...@apache.org> on 2020/01/05 00:56:36 UTC

[GitHub] [incubator-apisix] membphis commented on issue #1021: Modify the SSL module from ffi to ngx.ssl

membphis commented on issue #1021: Modify the SSL module from ffi to ngx.ssl
URL: https://github.com/apache/incubator-apisix/pull/1021#issuecomment-570833818
 
 
   @xiaobiaozhao Please fix the code style.
   
   https://travis-ci.org/apache/incubator-apisix/jobs/632655357#L698
   
   ```shell
   +./bin/apisix stop
   +sleep 1
   +make lint
   luacheck -q lua
   Checking lua/apisix/http/router/radixtree_sni.lua 2 warnings
       lua/apisix/http/router/radixtree_sni.lua:75:23: unused variable err
       lua/apisix/http/router/radixtree_sni.lua:85:23: unused variable err
   Total: 2 warnings / 0 errors in 70 files
   Makefile:71: recipe for target 'lint' failed
   make: *** [lint] Error 1
   ```

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services