| commit | c4f366a31e8b496d783ee6906ac3533738bb099c | [log] [tgz] |
|---|---|---|
| author | Trent Apted <tapted@chromium.org> | Tue May 16 15:32:48 2023 +1000 |
| committer | Chromeos LUCI <chromeos-scoped@luci-project-accounts.iam.gserviceaccount.com> | Tue May 23 02:54:57 2023 +0000 |
| tree | 7ad4b9d340a264343afbb15cf7a3a78ef875d1a8 | |
| parent | 2b864e17a3a059f06eb280cfd05ec12ee9041837 [diff] [blame] |
chromite: Split long strings on lines > 80 chars. Black will only split strings at spaces. We can tweak its `SPLIT_SAFE_CHARS` to split elsewhere - https://github.com/psf/black/blob/main/src/black/trans.py#L76 Allowing splits at "-","/", and "." lets it deal with a few more long strings in chromite automatically, so there will be fewer suppressions when the line-too-long lint is introduced. BUG=b:233893248 TEST=cros lint + CQ Change-Id: Ic092be1d0a80a4c13fac30d19756db715b42b8dd Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/chromite/+/4535497 Commit-Queue: Trent Apted <tapted@chromium.org> Reviewed-by: Jack Rosenthal <jrosenth@chromium.org> Tested-by: Trent Apted <tapted@chromium.org>
diff --git a/scripts/sysmon/git_metrics_unittest.py b/scripts/sysmon/git_metrics_unittest.py index bff7789..67be8a6 100644 --- a/scripts/sysmon/git_metrics_unittest.py +++ b/scripts/sysmon/git_metrics_unittest.py
@@ -22,7 +22,8 @@ def setUp(self): patcher = mock.patch( - "chromite.third_party.infra_libs.ts_mon.common.interface.state.store", + "chromite.third_party.infra_libs.ts_mon.common.interface.state." + "store", autospec=True, ) self.store = patcher.start()