Skip to content

Commit b3315d9

Browse files
committed
fix linting issues
1 parent d608b9a commit b3315d9

File tree

4 files changed

+24
-11
lines changed

4 files changed

+24
-11
lines changed

.gitignore

+1-1
Original file line numberDiff line numberDiff line change
@@ -63,4 +63,4 @@ web/src/__generated__
6363
Pulumi*.yaml
6464

6565
# performance profile files
66-
profiles
66+
profiles

api/graphql/filters.py

+18-6
Original file line numberDiff line numberDiff line change
@@ -89,8 +89,10 @@ def to_internal_filter_mapped(self, f: Callable[[T], Y]) -> GenericFilter[Y]:
8989
# issue in strawberry graphql where the usage of generics in input types causes major
9090
# slowdowns.
9191
# @see https://github.com/strawberry-graphql/strawberry/issues/3544
92-
@strawberry.input(description='Filter for GraphQL queries')
92+
@strawberry.input(description='String filter for GraphQL queries')
9393
class GraphQLFilterStr(GraphQLFilter[str]):
94+
"""String filter for GraphQL queries"""
95+
9496
eq: str | None = None
9597
in_: list[str] | None = None
9698
nin: list[str] | None = None
@@ -102,8 +104,10 @@ class GraphQLFilterStr(GraphQLFilter[str]):
102104
icontains: str | None = None
103105

104106

105-
@strawberry.input(description='Filter for GraphQL queries')
107+
@strawberry.input(description='Int filter for GraphQL queries')
106108
class GraphQLFilterInt(GraphQLFilter[int]):
109+
"""Int filter for GraphQL queries"""
110+
107111
eq: int | None = None
108112
in_: list[int] | None = None
109113
nin: list[int] | None = None
@@ -115,8 +119,10 @@ class GraphQLFilterInt(GraphQLFilter[int]):
115119
icontains: int | None = None
116120

117121

118-
@strawberry.input(description='Filter for GraphQL queries')
122+
@strawberry.input(description='Bool filter for GraphQL queries')
119123
class GraphQLFilterBool(GraphQLFilter[bool]):
124+
"""Bool filter for GraphQL queries"""
125+
120126
eq: bool | None = None
121127
in_: list[bool] | None = None
122128
nin: list[bool] | None = None
@@ -128,8 +134,10 @@ class GraphQLFilterBool(GraphQLFilter[bool]):
128134
icontains: bool | None = None
129135

130136

131-
@strawberry.input(description='Filter for GraphQL queries')
137+
@strawberry.input(description='Analysis status filter for GraphQL queries')
132138
class GraphQLFilterAnalysisStatus(GraphQLFilter[AnalysisStatus]):
139+
"""Analysis status filter for GraphQL queries"""
140+
133141
eq: AnalysisStatus | None = None
134142
in_: list[AnalysisStatus] | None = None
135143
nin: list[AnalysisStatus] | None = None
@@ -141,8 +149,10 @@ class GraphQLFilterAnalysisStatus(GraphQLFilter[AnalysisStatus]):
141149
icontains: AnalysisStatus | None = None
142150

143151

144-
@strawberry.input(description='Filter for GraphQL queries')
152+
@strawberry.input(description='Datetime filter for GraphQL queries')
145153
class GraphQLFilterDatetime(GraphQLFilter[datetime.datetime]):
154+
"""Datetime filter for GraphQL queries"""
155+
146156
eq: datetime.datetime | None = None
147157
in_: list[datetime.datetime] | None = None
148158
nin: list[datetime.datetime] | None = None
@@ -154,8 +164,10 @@ class GraphQLFilterDatetime(GraphQLFilter[datetime.datetime]):
154164
icontains: datetime.datetime | None = None
155165

156166

157-
@strawberry.input(description='Filter for GraphQL queries')
167+
@strawberry.input(description='Date filter for GraphQL queries')
158168
class GraphQLFilterDate(GraphQLFilter[datetime.date]):
169+
"""Date filter for GraphQL queries"""
170+
159171
eq: datetime.date | None = None
160172
in_: list[datetime.date] | None = None
161173
nin: list[datetime.date] | None = None

api/server.py

+4-3
Original file line numberDiff line numberDiff line change
@@ -59,15 +59,16 @@ async def app_lifespan(_: FastAPI):
5959
from pyinstrument import Profiler
6060
from pyinstrument.renderers.speedscope import SpeedscopeRenderer
6161

62-
@app.middleware("http")
62+
@app.middleware('http')
6363
async def profile_request(request: Request, call_next):
64-
profiler = Profiler(async_mode="enabled")
64+
"""Middleware to allow profiling of requests to the api"""
65+
profiler = Profiler(async_mode='enabled')
6566
profiler.start()
6667
resp = await call_next(request)
6768
profiler.stop()
6869

69-
text_output = profiler.output_text()
7070
if 'text' in PROFILE_REQUESTS_OUTPUT:
71+
text_output = profiler.output_text()
7172
print(text_output)
7273

7374
timestamp = datetime.datetime.now().replace(microsecond=0).isoformat()

docs/installation.md

+1-1
Original file line numberDiff line numberDiff line change
@@ -429,4 +429,4 @@ If you are working on performance issues it can be handy to generate a report th
429429

430430
There are a few different options for outputting profiles which can be specified in the `SM_PROFILE_REQUESTS_OUTPUT` environment variable. The possible values are `text` which will print the profiling results to stdout, `html` which will generate an interactive pyinstrument report, or `json` which will generate a json profiling report which can be dropped into [speedscope](https://www.speedscope.app/) to explore the profile.
431431

432-
You can output multiple report types by specifying the types in a list like: `export SM_PROFILE_REQUESTS_OUTPUT=json,text,html`
432+
You can output multiple report types by specifying the types in a list like: `export SM_PROFILE_REQUESTS_OUTPUT=json,text,html`

0 commit comments

Comments
 (0)