dcslin commented on issue #599: Enabled linting check in CI
URL: https://github.com/apache/singa/pull/599#issuecomment-589029524
 
 
   currently the linting.sh check is checking below files:
   ```
   find python/ \
       examples/autograd \
       test/python/ -iname *.py | xargs pylint
   
   find src/api/ \
       src/core/ \
       src/proto/ \
       src/utils/ \
       include/singa/core/ \
       include/singa/utils/ \
       src/model/operation/ \
       include/singa/io/communicator.h \
       src/io/communicator.cc \
       test/singa/ -iname *.h -o -iname *.cc | xargs cpplint --quiet
   ```
   however the problem is there are still a lot of warnings. One way is to 
customize the linting config. Other way we define the linting check is for 
reference and not compulsary.

----------------------------------------------------------------
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:
[email protected]


With regards,
Apache Git Services

Reply via email to