Skip to content

Conversation

ruthvik-17
Copy link
Contributor

Proposed changes

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.

@ruthvik-17 ruthvik-17 requested a review from a team as a code owner September 18, 2025 07:33
@github-actions github-actions bot added documentation Improvements or additions to documentation product/nim Issues related to NGINX Instance Manager labels Sep 18, 2025
@github-actions
Copy link

github-actions bot commented Sep 18, 2025

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

@ruthvik-17
Copy link
Contributor Author

ruthvik-17 commented Sep 18, 2025

@ruthvik-17
Copy link
Contributor Author

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

Copy link
Member

@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.

Happy to approve once the banner shortcode is fixed.

Copy link
Member

@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.

Happy to approve once the call-out adjustments have been made.

Only the call-out type is a required parameter: there is no need to add placeholders for unused parameters.

If there is an issue with content formatting, this should be reported as a bug, not worked around.

@ADubhlaoich ADubhlaoich merged commit 1c74f0d into nginx:main Sep 22, 2025
2 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.

2 participants