JDevlieghere added inline comments.
================ Comment at: lldb/test/API/functionalities/interactive_scripted_process/interactive_scripted_process.py:9 + +import os,json,struct,signal +import time ---------------- bulbazord wrote: > mib wrote: > > bulbazord wrote: > > > nit: import these all on different lines > > why :p ? > I don't actually think it matters a ton, which is why it was a nit, but I > think there's something to be said for consistency with the rest of lldb. Given the discussion in https://discourse.llvm.org/t/rfc-document-and-standardize-python-code-style, let's format new tests with `black`. It doesn't look like the tool has a preference between those two so I would vote for consistency. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D145297/new/ https://reviews.llvm.org/D145297 _______________________________________________ lldb-commits mailing list lldb-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
