[Lldb-commits] [lldb] [lldb] Format more Python files with black (PR #65979)

https://github.com/DavidSpickett closed https://github.com/llvm/llvm-project/pull/65979 ___ lldb-commits mailing list lldb-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

[Lldb-commits] [lldb] [lldb] Format more Python files with black (PR #65979)

https://github.com/DavidSpickett updated https://github.com/llvm/llvm-project/pull/65979: >From 251eaf5271db6ef9646f843ef037d54406544f8d Mon Sep 17 00:00:00 2001 From: David Spickett Date: Mon, 11 Sep 2023 16:52:05 + Subject: [PATCH] [lldb] Format more Python files with black By running th

[Lldb-commits] [lldb] [lldb] Format more Python files with black (PR #65979)

https://github.com/JDevlieghere approved this pull request. LGTM. I formatted the test, examples and script subdirectory. This is more comprehensive. https://github.com/llvm/llvm-project/pull/65979 ___ lldb-commits mailing list lldb-commits@lists.llv

[Lldb-commits] [lldb] [lldb] Format more Python files with black (PR #65979)

DavidSpickett wrote: I looked around for a way to exclude dirs from black but unfortunately the way is `--exclude`. I was hoping to place some settings file in third_party/ to disable it. https://github.com/llvm/llvm-project/pull/65979 ___ lldb-commi

[Lldb-commits] [lldb] [lldb] Format more Python files with black (PR #65979)

llvmbot wrote: @llvm/pr-subscribers-lldb Changes By running this from lldb/ $ black --exclude "third_party/|scripts/|utils/" ./ -- Patch is 21.03 KiB, truncated to 20.00 KiB below, full version: https://github.com/llvm/llvm-project/pull/65979.diff 9 Files Affected: - (modified) lldb/bindi

[Lldb-commits] [lldb] [lldb] Format more Python files with black (PR #65979)

DavidSpickett wrote: Putting this into review mainly to ask @JDevlieghere if any of these shouldn't be formatted. Third party obviously we don't want to format that. Scripts and utils I don't see why not but maybe there are downstream concerns? https://github.com/llvm/llvm-project/pull/65979

[Lldb-commits] [lldb] [lldb] Format more Python files with black (PR #65979)

https://github.com/llvmbot labeled https://github.com/llvm/llvm-project/pull/65979 ___ lldb-commits mailing list lldb-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

[Lldb-commits] [lldb] [lldb] Format more Python files with black (PR #65979)

https://github.com/DavidSpickett review_requested https://github.com/llvm/llvm-project/pull/65979 ___ lldb-commits mailing list lldb-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

[Lldb-commits] [lldb] [lldb] Format more Python files with black (PR #65979)

https://github.com/DavidSpickett created https://github.com/llvm/llvm-project/pull/65979: By running this from lldb/ $ black --exclude "third_party/|scripts/|utils/" ./ >From 56cea5795e0274fe7672c727340330cf46aa30a4 Mon Sep 17 00:00:00 2001 From: David Spickett Date: Mon, 11 Sep 2023 16:52:05

[Lldb-commits] [lldb] [lldb] Format more Python files with black (PR #65979)

https://github.com/DavidSpickett review_requested https://github.com/llvm/llvm-project/pull/65979 ___ lldb-commits mailing list lldb-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits