Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: Flask to FastAPI #701

Merged
merged 13 commits into from
May 4, 2023
Merged

feat: Flask to FastAPI #701

merged 13 commits into from
May 4, 2023

Conversation

sbrunato
Copy link
Collaborator

@sbrunato sbrunato commented Apr 6, 2023

Moves server mode from Flask to FastAPI

Fixes #627
Fixes #628
Fixes #718

Tests skipped if AF_UNIX socket is not available in order to prevent pytest_socket.SocketBlockedError in Windows.
See miketheman/pytest-socket#54 and python/cpython#77589

@github-actions
Copy link
Contributor

github-actions bot commented Apr 6, 2023

Test Results

       4 files  ±  0         4 suites  ±0   3m 49s ⏱️ -33s
   387 tests +  8     385 ✔️ +11    2 💤 ±  0  0  - 3 
1 548 runs  +83  1 492 ✔️ +40  56 💤 +46  0  - 3 

Results for commit 2325807. ± Comparison against base commit c328060.

♻️ This comment has been updated with latest results.

@sbrunato sbrunato force-pushed the fastapi branch 2 times, most recently from 0745945 to e859544 Compare April 7, 2023 15:19
@sbrunato sbrunato force-pushed the fastapi branch 2 times, most recently from 79c2d27 to e6c7a7c Compare April 18, 2023 14:38
@github-actions
Copy link
Contributor

github-actions bot commented May 3, 2023

Code Coverage (Ubuntu)

File Coverage
All files 82%

Minimum allowed coverage is 70%

Generated by 🐒 cobertura-action against 2325807

@github-actions
Copy link
Contributor

github-actions bot commented May 3, 2023

Code Coverage (Windows)

File Coverage
All files 76%

Minimum allowed coverage is 70%

Generated by 🐒 cobertura-action against 2325807

@CS-SI CS-SI deleted a comment from github-actions bot May 4, 2023
@sbrunato sbrunato marked this pull request as ready for review May 4, 2023 14:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

The port is missing in STAC API web page. plugins pre-build in server mode move to FastAPI and orjson
1 participant