-
Notifications
You must be signed in to change notification settings - Fork 82
Simplify readability for NGINX Plus Admin Guide: Gateway with uWSGI and Django #518
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
Conversation
🎉 Thank you for your contribution! It appears you have not yet signed the F5 Contributor License Agreement (CLA), which is required for your changes to be incorporated into an F5 Open Source Software (OSS) project. Please kindly read the F5 CLA and reply on a new comment with the following text to agree: I have hereby read the F5 CLA and agree to its terms 2 out of 3 committers have signed the CLA. |
I have hereby read the F5 CLA and agree to its terms |
recheck |
1 similar comment
recheck |
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 love the improvements! I have a couple of suggested changes. If you'd like to talk about it while we're at the conference, let me know!
content/nginx/admin-guide/web-server/app-gateway-uwsgi-django.md
Outdated
Show resolved
Hide resolved
content/nginx/admin-guide/web-server/app-gateway-uwsgi-django.md
Outdated
Show resolved
Hide resolved
recheck |
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 so much!
Congratulations @kmjaimes on your first time PR! (I see the test for the CLA is still failing. But we see your acceptance in the comments, so I'm overriding the test, and accepting your PR!) |
Thanks for the opportunity! It's been a great experience and I look forward to contributing in the future. |
Proposed changes
Problem: Article is listed as "difficult" to read.
Solution: Use Hemingway App to drop readability score from 10 to 9. Fleisch Kinkaid estimate from 14.46 to 8.2.
If this PR addresses an issue on GitHub, ensure that you link to it here: #359
Closes Issue #359