Skip to content

Conversation

vvnginx
Copy link
Contributor

@vvnginx vvnginx commented Jul 30, 2025

Proposed changes

"Updated manual install steps of NIM for RHEL-9 distro. Updated NAP compiler version in offline installation steps "

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.

@vvnginx vvnginx requested a review from a team as a code owner July 30, 2025 12:38
@github-actions github-actions bot added documentation Improvements or additions to documentation product/nim Issues related to NGINX Instance Manager labels Jul 30, 2025
Copy link
Contributor

@ADubhlaoich ADubhlaoich left a comment

Choose a reason for hiding this comment

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

LGTM!

@vvnginx vvnginx force-pushed the patch-2 branch 2 times, most recently from d9c7f15 to adbd504 Compare August 6, 2025 10:10
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.

Discussion in progress with @vvnginx . I'll likely pull my hold once he updates this PR.

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 for the latest update. Approving and proceeding with merge

@mjang mjang merged commit 9c05ab0 into nginx:main Aug 12, 2025
7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation product/nim Issues related to NGINX Instance Manager
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants