-
-
Notifications
You must be signed in to change notification settings - Fork 227
docs: Remove openapi generation python2 compatibility claim #499
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
docs: Remove openapi generation python2 compatibility claim #499
Conversation
remove python compatibility claim
add note about being written in python
fix double Gg, add periods
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks @rtaycher!
I've had a few folks ask for an update here, thanks for kick starting it. I rephrased to focus on the positives of this generator instead of a comparison so it won't fall out of date again. What do you think?
I'm also thinking about adding a note around extensive testing... the openapitools generator has accidentally introduced breaking changes for me more times than I can count 😅
Codecov Report
@@ Coverage Diff @@
## main #499 +/- ##
=========================================
Coverage 100.00% 100.00%
=========================================
Files 48 48
Lines 1609 1609
=========================================
Hits 1609 1609 Continue to review full report at Codecov.
|
Co-authored-by: Dylan Anthony <[email protected]>
fix number in numbered list
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I included the note about Jinja2. Thanks!
Seems like its slightly outdated according to #242
I haven't played enough with both for a deeper comparison but I think it would be nice to update the basics at least