pelikan icon indicating copy to clipboard operation
pelikan copied to clipboard

choose a linter/formatting tool

Open thinkingfish opened this issue 9 years ago • 5 comments

A few things to consider:

  • availability across all platforms we plan to support;
  • customized rules;
  • IDE/editor integration;
  • build integration.

thinkingfish avatar Nov 10 '15 03:11 thinkingfish

@seppo0010 @kevyang @sagar0 continuing our discussion earlier...

thinkingfish avatar Nov 10 '15 03:11 thinkingfish

Candidates so far:

  • http://astyle.sourceforge.net/
  • http://clang.llvm.org/docs/ClangFormat.html

Please review/add/comment

thinkingfish avatar Nov 10 '15 03:11 thinkingfish

Both astyle and clang-format are available in brew and aptitude. Either of them works for me.

seppo0010 avatar Nov 10 '15 03:11 seppo0010

I recommend that we use clang-format mainly due to better IDE/editor integration.

Will create a style config file specifying rules according to our style guide.

thinkingfish avatar Nov 26 '15 07:11 thinkingfish

Can you also please include in the readme how to install it and how to integrate with the editor of your choice (hopefully vim).

seppo0010 avatar Nov 26 '15 17:11 seppo0010