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鈥檒l occasionally send you account related emails.
Already on GitHub? Sign in to your account
馃摑 Add external link to "FastAPI for Flask Users" #2280
Conversation
馃摑 Docs preview for commit 5ec2a34 at: https://5f9e8ff8a392bfc00adfcd35--fastapi.netlify.app |
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.
We do not have an alphabetical order. The order is first-come-first-serve, so to be fair, please add this to the end.
@ycd I've migrated it to the end of the list. Should adhere to the FCFS rule now. |
Codecov Report
@@ Coverage Diff @@
## master #2280 +/- ##
=========================================
Coverage 100.00% 100.00%
=========================================
Files 240 240
Lines 7101 7101
=========================================
Hits 7101 7101 Continue to review full report at Codecov.
|
馃摑 Docs preview for commit c7de6f2 at: https://5f9fa7c380012668801451a6--fastapi.netlify.app |
馃摑 Docs preview for commit a748ba1 at: https://5fba5d3f2c6b3e94a6fe348c--fastapi.netlify.app |
@ycd I've fixed the merge conflicts. Can you look into it now? |
Thanks, actually, you don't have to change the order, since your PR is older than the latest one, FYI. |
@ycd I've no problems with the ordering. Let's merge it the way it is. |
I have added a link to my article FastAPI for Flask Users in the external links section of documentation.