CodeFactor で行われる cpplint.py のチェックをカスタマイズする基盤を作る #946
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
https://github.com/google/styleguide/blob/6271f3f473ceb3a7fef99388a3040903b1a145f1/cpplint/cpplint.py#L156-L197 https://github.com/google/styleguide/tree/gh-pages/cpplint https://github.com/codefactor-io/default-configs/blob/master/CPPLINT.cfg
m-tmatma
added
CI
berryzplus
approved these changes
Jun 8, 2019
m-tmatma
changed the title
HoppingTappy
pushed a commit
to HoppingTappy/sakura
that referenced
this pull request
Jun 11, 2019
CodeFactor で行われる cppcheck.py のチェックをカスタマイズする基盤を作る
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
PR の目的
CodeFactor で行われる cpplint.py のチェックを明示的に行うようにして
容易にカスタマイズできるようにする。
カテゴリ
PR の背景
CodeFactor で行われるスタイルチェックのうちスペースや空行に関するものがあるが
修正する必要のないものに対して、codefactor 上でいちいち無効化するのは面倒。
#724 のように修正しないと決めたものを無視できるようにするための基盤を作る。
この PR 自体では検出される警告数に違いはなく、現状の CodeFactor の cpp のチェックと
同等の条件とする。。
PR のメリット
CodeFactor の警告を自動的に無視するための指定を行うための基盤ができる。
PR のデメリット (トレードオフとかあれば)
なし
PR の影響範囲
CodeFactor
関連チケット
#724
close #945 (一部警告を無視できるかの実験。効果を確認できた)
参考資料