mirror of
https://github.com/ROCm/ROCm.git
synced 2026-02-21 03:00:39 -05:00
I've add an option to yapf to do what we want for long lines, see https://github.com/google/yapf/pull/1177. We can now have a real Python formatter, yay! To make this PR, I ran my modified yapf over the repository, then looked over the full diff. Where yapf was mangling the param list of long function decls/calls (mostly kernels), I manually added `#` to put linebreaks where we want. I fixed up other formatting too -- mostly adding or removing a trailing comma from lists. Overall, trailing `#` was sufficient to get formatting similar to our current code. I didn't have to disable yapf anywhere. --------- Co-authored-by: Phil Tillet <phil@openai.com>
27 lines
790 B
TOML
27 lines
790 B
TOML
[build-system]
|
|
requires = ["setuptools>=40.8.0", "wheel", "cmake>=3.18", "ninja>=1.11.1"]
|
|
|
|
[tool.yapf]
|
|
based_on_style = "pep8"
|
|
column_limit = 120
|
|
disable_split_list_with_comment = true
|
|
each_dict_entry_on_separate_line=false
|
|
split_before_named_assigns = false
|
|
split_complex_comprehension = true
|
|
|
|
[tool.yapfignore]
|
|
ignore_patterns = [
|
|
# This exclusion is also specified in .pre-commit-config.yaml.
|
|
# - We put it here because if you run yapf directly, we want it to skip the
|
|
# file.
|
|
# - We also put it in .pre-commit-config because yapf raises an error if
|
|
# pre-commit runs it but all of the files it might touch are ignored!
|
|
"python/test/unit/language/test_line_info.py"
|
|
]
|
|
|
|
[tool.ruff]
|
|
line-length = 120
|
|
|
|
[tool.ruff.lint]
|
|
ignore = ["E501", "E701", "E731", "E741"]
|