Skip to content

Conversation

cs168898
Copy link
Contributor

@cs168898 cs168898 commented Sep 3, 2025

Fixed the broken Docker file link that was mentioned in nginx/agent#1087 (Issue #1087)
NGINX Documentation Issue number: #1051

Proposed changes

The pull request fixes the URL link in the NGINX Agent documentation Dockerfile URL to the updated URL.

I changed it through clicking the 'edit page' button located at the bottom of the documentation page then editing it in the forked version before committing it and creating a PR.

The changes will not affect anything else but it might be affected in the future if the structure of the NGINX Agent repository changes again.

Additional Context:
I am new to the world of contributing, please let me know if I am doing anything wrongly, thank you!

Checklist

Before sharing this pull request, I completed the following checklist:

Footnotes

  1. Potentially sensitive information includes personally identify information (PII), authentication credentials, and live URLs. Refer to the style guide for guidance about placeholder content.

Fixed the broken Docker file link that was mentioned in nginx/agent#1087
(Issue #1087)
@cs168898 cs168898 requested a review from a team as a code owner September 3, 2025 07:22
@github-actions github-actions bot added documentation Improvements or additions to documentation product/agent Issues related to NGINX Agent labels Sep 3, 2025
Copy link

github-actions bot commented Sep 3, 2025

✅ All required contributors have signed the F5 CLA for this PR. Thank you!
Posted by the CLA Assistant Lite bot.

@cs168898
Copy link
Contributor Author

cs168898 commented Sep 3, 2025

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

@mjang mjang added the community label Sep 3, 2025
@mjang
Copy link
Contributor

mjang commented Sep 3, 2025

I have a 2nd implied approval internally, so I'm proceeding with merge.

@mjang
Copy link
Contributor

mjang commented Sep 3, 2025

thank you @cs168898 so much for your contribution!

@mjang mjang merged commit 177766b into nginx:main Sep 3, 2025
7 checks passed
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/agent Issues related to NGINX Agent
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants