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
rafsaf · May 23, 2024 · ba24f1a · ba24f1a
2 parents ea9a261 + 5b51e5d
commit ba24f1a
Show file tree

Hide file tree

Showing 3 changed files with 6 additions and 1 deletion.
diff --git a/.github/workflows/tests.yml b/.github/workflows/tests.yml
@@ -5,6 +5,8 @@ on:
       - "**"
     tags-ignore:
       - "*.*"
+  pull_request:
+    types: [opened, reopened]
 
 jobs:
   tests:

diff --git a/.github/workflows/type_check.yml b/.github/workflows/type_check.yml
@@ -5,6 +5,8 @@ on:
       - "**"
     tags-ignore:
       - "*.*"
+  pull_request:
+    types: [opened, reopened]
 
 jobs:
   type_check:

diff --git a/app/main.py b/app/main.py
@@ -20,7 +20,8 @@
 app.add_middleware(
     CORSMiddleware,
     allow_origins=[
-        str(origin).rstrip("/") for origin in get_settings().security.backend_cors_origins
+        str(origin).rstrip("/")
+        for origin in get_settings().security.backend_cors_origins
     ],
     allow_credentials=True,
     allow_methods=["*"],