浏览代码

Merge pull request #258 from twiest/pr

Added 'stylistic exception' to the best practices guide.
Thomas Wiest 10 年之前
父节点
当前提交
f906869478
共有 1 个文件被更改,包括 1 次插入0 次删除
  1. 1 0
      docs/best_practices_guide.adoc

+ 1 - 0
docs/best_practices_guide.adoc

@@ -49,6 +49,7 @@ Instead, http://docs.pylint.org/faq.html#is-it-possible-to-locally-disable-a-par
 .Exceptions:
 1. When PyLint fails because of a dependency that can't be installed on the build bot
 1. When PyLint fails because of including a module that is outside of control (like Ansible)
+1. When PyLint fails, but the code makes more sense the way it is formatted (stylistic exception). For this exception, the description of the PyLint disable MUST state why the code is more clear, AND the person reviewing the PR will decide if they agree or not. The reviewer may reject the PR if they disagree with the reason for the disable.
 
 '''
 [cols="2v,v"]