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

Added Pyotr to application frameworks #51

Merged
merged 4 commits into from
Oct 29, 2021
Merged

Added Pyotr to application frameworks #51

merged 4 commits into from
Oct 29, 2021

Conversation

berislavlopac
Copy link
Contributor

@berislavlopac berislavlopac commented Oct 26, 2020

Checklist

  • This project is explicitly related to ASGI.
  • The new list entry contains a project name, URL and description.

What is this project?

Pyotr is a helpful library, built on top of Starlette and HTTPX, that includes both server and client side component using an OpenAPI specification for routing and validation.

Do you know about other similar projects?

The closest I am aware of were some attempts to add support for Quart to connexion, but that is all. On the client side, the only similar project is Bravado, which is a) barely maintained, and b) not asynchronous.

If so, how is this one different?

Full ASGI support, meaning that it can be used as any other ASGI app; plus the full asynchronous client-side support.


Anyone who agrees with this pull request can add a +1.

@florimondmanca florimondmanca added the please vote Vote to confirm interest in adding this project to the list label Oct 26, 2020
Copy link
Owner

@florimondmanca florimondmanca left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks! Happy to consider it. Leaving this open for a bit, in the meantime could you please update the PR description following the PR template for new entries? Not willing to be that guy, but having entry suggestions follow the same template helps a bit in reviewing what's obviously nice to have in the list, and what's less clear. :-) (And if you have suggestions on the template itself — by all means feel free to share them.)

README.md Outdated Show resolved Hide resolved
Co-authored-by: Florimond Manca <florimond.manca@gmail.com>
@berislavlopac
Copy link
Contributor Author

Thank you, just updated it - thank you for the clarification. I don't have additional ideas for the template, sorry!

@florimondmanca
Copy link
Owner

Hi @berislavlopac,

I'm reviewing this again just now, and I see the documentation reads:

WARNING: This is still very much work in progress and not nearly ready for any kind of production.

Although I think this project fills an interesting space (OpenAPI focus), do you feel comfortable listing this here given the project status? We may also add a note in the project description.

@florimondmanca florimondmanca added candidate Project is relevant to include in the list and removed please vote Vote to confirm interest in adding this project to the list labels Oct 18, 2021
@berislavlopac
Copy link
Contributor Author

Hey @florimondmanca, thanks for the message! Sure, I'd like to have it included, ideally with a note. I'm hoping to spend some more time on it in the near future. 😊

@florimondmanca
Copy link
Owner

@berislavlopac Fair enough, let's get it in as-is, I think. :-)

@florimondmanca florimondmanca merged commit ec7c0f7 into florimondmanca:master Oct 29, 2021
@berislavlopac berislavlopac deleted the pyotr branch October 30, 2021 16:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
candidate Project is relevant to include in the list
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants