Skip to content
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

feat: Fix instruction link for NGINX Ingress Helm Operator #3596

Merged
merged 9 commits into from Mar 16, 2023

Conversation

ADubhlaoich
Copy link
Contributor

@ADubhlaoich ADubhlaoich commented Feb 27, 2023

This commit makes a small change to the the third prerequisite in the installation document to point at the correct instructions it is referencing.

It also expands the link itself to make it clearer what precisely the reader is being referenced to, and prefixes F5 to the first instance of the product name (Ignoring the ephemeral note).

Proposed changes

Describe the use case and detail of the change. If this PR addresses an issue on GitHub, make sure to include a link to that issue here in this description (not in the title of the PR).

Checklist

Before creating a PR, run through this checklist and mark each as complete.

  • I have read the CONTRIBUTING doc
  • I have added tests that prove my fix is effective or that my feature works
  • I have checked that all unit tests pass after adding my changes
  • I have updated necessary documentation
  • I have rebased my branch onto main
  • I will ensure my PR is targeting the main branch and pulling from my branch from my own fork

This commit makes a small change to the the third prerequisite in
the installation document to point at the correct instructions
it is referencing.

It also expands the link itself to make it clearer what precisely
the reader is being referenced to, and prefixes F5 to the first
instance of the product name (Ignoring the ephemeral note).
@ADubhlaoich ADubhlaoich requested a review from a team as a code owner February 27, 2023 15:49
@ADubhlaoich ADubhlaoich self-assigned this Feb 27, 2023
@github-actions github-actions bot added the documentation Pull requests/issues for documentation label Feb 27, 2023
ADubhlaoich and others added 2 commits February 28, 2023 11:38
Co-authored-by: Luca Comellini <luca.com@gmail.com>
Signed-off-by: Alan Dooley <ADubhlaoich@users.noreply.github.com>
@codecov-commenter
Copy link

codecov-commenter commented Feb 28, 2023

Codecov Report

Merging #3596 (05d14a7) into main (97ef3c3) will not change coverage.
The diff coverage is n/a.

❗ Current head 05d14a7 differs from pull request most recent head d2f21e3. Consider uploading reports for the commit d2f21e3 to get more accurate results

📣 This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more

@@           Coverage Diff           @@
##             main    #3596   +/-   ##
=======================================
  Coverage   52.23%   52.23%           
=======================================
  Files          59       59           
  Lines       16877    16877           
=======================================
  Hits         8816     8816           
  Misses       7766     7766           
  Partials      295      295           

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@lucacome lucacome merged commit eaf883f into main Mar 16, 2023
51 of 52 checks passed
@lucacome lucacome deleted the docs/fix-operator-link branch March 16, 2023 17:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Pull requests/issues for documentation
Projects
Status: Done 🚀
Development

Successfully merging this pull request may close these issues.

None yet

7 participants