Skip to content

Commit ba24f1a

Browse files
authored
Merge pull request #62 from rafsaf/add-on-pull-request-to-ci
fix lint issue, add on:pull_request in CI to handle PR created on github
2 parents ea9a261 + 5b51e5d commit ba24f1a

File tree

3 files changed

+6
-1
lines changed

3 files changed

+6
-1
lines changed

.github/workflows/tests.yml

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -5,6 +5,8 @@ on:
55
- "**"
66
tags-ignore:
77
- "*.*"
8+
pull_request:
9+
types: [opened, reopened]
810

911
jobs:
1012
tests:

.github/workflows/type_check.yml

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -5,6 +5,8 @@ on:
55
- "**"
66
tags-ignore:
77
- "*.*"
8+
pull_request:
9+
types: [opened, reopened]
810

911
jobs:
1012
type_check:

app/main.py

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -20,7 +20,8 @@
2020
app.add_middleware(
2121
CORSMiddleware,
2222
allow_origins=[
23-
str(origin).rstrip("/") for origin in get_settings().security.backend_cors_origins
23+
str(origin).rstrip("/")
24+
for origin in get_settings().security.backend_cors_origins
2425
],
2526
allow_credentials=True,
2627
allow_methods=["*"],

0 commit comments

Comments
 (0)