diff options
author | Andrew Morgan <1342360+anoadragon453@users.noreply.github.com> | 2020-07-20 16:43:49 -0400 |
---|---|---|
committer | GitHub <noreply@github.com> | 2020-07-20 21:43:49 +0100 |
commit | b7ddece2a6d541367a10d00d1e7851e29fce2612 (patch) | |
tree | 25bc994728d99e19e1bd68e430c1e7907f276773 /contrib/graph/graph.py | |
parent | Remove unused code from synapse.logging.utils. (#7897) (diff) | |
download | synapse-b7ddece2a6d541367a10d00d1e7851e29fce2612.tar.xz |
Lint the contrib/ directory in CI and linting scripts, add synctl to linting script (#7914)
Run `isort`, `flake8` and `black` over the `contrib/` directory and `synctl` script. The latter was already being done in CI, but now the linting script does it too. Fixes https://github.com/matrix-org/synapse/issues/7910
Diffstat (limited to 'contrib/graph/graph.py')
-rw-r--r-- | contrib/graph/graph.py | 21 |
1 files changed, 10 insertions, 11 deletions
diff --git a/contrib/graph/graph.py b/contrib/graph/graph.py index 92736480eb..de33fac1c7 100644 --- a/contrib/graph/graph.py +++ b/contrib/graph/graph.py @@ -1,5 +1,13 @@ from __future__ import print_function +import argparse +import cgi +import datetime +import json + +import pydot +import urllib2 + # Copyright 2014-2016 OpenMarket Ltd # # Licensed under the Apache License, Version 2.0 (the "License"); @@ -15,15 +23,6 @@ from __future__ import print_function # limitations under the License. -import sqlite3 -import pydot -import cgi -import json -import datetime -import argparse -import urllib2 - - def make_name(pdu_id, origin): return "%s@%s" % (pdu_id, origin) @@ -33,7 +32,7 @@ def make_graph(pdus, room, filename_prefix): node_map = {} origins = set() - colors = set(("red", "green", "blue", "yellow", "purple")) + colors = {"red", "green", "blue", "yellow", "purple"} for pdu in pdus: origins.add(pdu.get("origin")) @@ -49,7 +48,7 @@ def make_graph(pdus, room, filename_prefix): try: c = colors.pop() color_map[o] = c - except: + except Exception: print("Run out of colours!") color_map[o] = "black" |