Skip to content

Commit

Permalink
pydrake: Mark explicit clang-format enforcement
Browse files Browse the repository at this point in the history
Now all packages explicitly list whether they enable the clang-format
idempotency linter or not.  Only //bindings/pydrake/common currently has
it disabled (with a TODO).
  • Loading branch information
jwnimmer-tri committed Nov 30, 2018
1 parent 653f413 commit 3b9ef87
Show file tree
Hide file tree
Showing 3 changed files with 13 additions and 3 deletions.
5 changes: 4 additions & 1 deletion bindings/BUILD.bazel
Original file line number Diff line number Diff line change
Expand Up @@ -26,4 +26,7 @@ drake_pybind_library(
package_info = get_bazel_workaround_4594_libdrake_package_info(),
)

add_lint_tests()
add_lint_tests(
cpplint_data = ["//bindings/pydrake:.clang-format"],
enable_clang_format_lint = True,
)
6 changes: 5 additions & 1 deletion bindings/pydrake/common/BUILD.bazel
Original file line number Diff line number Diff line change
Expand Up @@ -433,4 +433,8 @@ drake_py_unittest(
],
)

add_lint_tests()
add_lint_tests(
cpplint_data = ["//bindings/pydrake:.clang-format"],
# TODO(jwnimmer-tri) We should enable this eventually.
enable_clang_format_lint = False,
)
5 changes: 4 additions & 1 deletion bindings/pydrake/util/BUILD.bazel
Original file line number Diff line number Diff line change
Expand Up @@ -102,4 +102,7 @@ drake_py_test(
deps = [":util"],
)

add_lint_tests()
add_lint_tests(
cpplint_data = ["//bindings/pydrake:.clang-format"],
enable_clang_format_lint = True,
)

0 comments on commit 3b9ef87

Please sign in to comment.