Skip to content

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

Merged
merged 5 commits into from
May 5, 2025

Conversation

kmjaimes
Copy link
Contributor

@kmjaimes kmjaimes commented May 4, 2025

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

@kmjaimes kmjaimes requested a review from a team as a code owner May 4, 2025 22:43
Copy link

github-actions bot commented May 4, 2025

🎉 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.
✅ (kmjaimes)[https://github.com/kmjaimes]
✅ (mjang)[https://github.com/mjang]
@kristen Jaimes
Kristen Jaimes seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.
You can retrigger this bot by commenting recheck in this Pull Request. Posted by the CLA Assistant Lite bot.

@github-actions github-actions bot added documentation Improvements or additions to documentation product/nginx-plus NGINX Plus (and NGINX Open Source in docs.ngninx.com) labels May 4, 2025
@kmjaimes
Copy link
Contributor Author

kmjaimes commented May 4, 2025

I have hereby read the F5 CLA and agree to its terms

@kmjaimes
Copy link
Contributor Author

kmjaimes commented May 4, 2025

recheck

1 similar comment
@mjang
Copy link
Contributor

mjang commented May 4, 2025

recheck

Copy link
Contributor

@mjang mjang left a 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!

@mjang
Copy link
Contributor

mjang commented May 5, 2025

recheck

Copy link
Contributor

@mjang mjang left a comment

Choose a reason for hiding this comment

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

Thanks so much!

@mjang
Copy link
Contributor

mjang commented May 5, 2025

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!)

@mjang mjang merged commit c53026f into nginx:main May 5, 2025
5 of 6 checks passed
@mjang mjang added the community label May 6, 2025
@kmjaimes
Copy link
Contributor Author

kmjaimes commented May 6, 2025

Thanks for the opportunity! It's been a great experience and I look forward to contributing in the future.

@kmjaimes kmjaimes deleted the wtd-issue-359 branch May 6, 2025 21:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
community documentation Improvements or additions to documentation product/nginx-plus NGINX Plus (and NGINX Open Source in docs.ngninx.com)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants