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

Update installation-with-manifests.md #4090

Merged
merged 2 commits into from
Jul 12, 2023
Merged

Update installation-with-manifests.md #4090

merged 2 commits into from
Jul 12, 2023

Conversation

jputrino
Copy link
Contributor

@jputrino jputrino commented Jul 11, 2023

Fixes a typo in the GlobalConfiguration CRD step.

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

Fixes a typo in the GlobalConfiguration CRD step.

Signed-off-by: Jodie Putrino <[email protected]>
@jputrino jputrino requested a review from a team as a code owner July 11, 2023 17:57
@github-actions github-actions bot added the documentation Pull requests/issues for documentation label Jul 11, 2023
@github-actions
Copy link

github-actions bot commented Jul 11, 2023

Dependency Review

✅ No vulnerabilities or license issues found.

Scanned Manifest Files

@codecov
Copy link

codecov bot commented Jul 11, 2023

Codecov Report

Merging #4090 (0fc54e7) into main (82e6752) will increase coverage by 0.02%.
The diff coverage is n/a.

❗ Current head 0fc54e7 differs from pull request most recent head d2243f4. Consider uploading reports for the commit d2243f4 to get more accurate results

@@            Coverage Diff             @@
##             main    #4090      +/-   ##
==========================================
+ Coverage   51.84%   51.87%   +0.02%     
==========================================
  Files          59       59              
  Lines       16710    16710              
==========================================
+ Hits         8664     8668       +4     
+ Misses       7747     7745       -2     
+ Partials      299      297       -2     

see 1 file with indirect coverage changes

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

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.

I suspect this was a case of me intending to update an instance of "the Ingress Controller" to "NGINX Ingress Controller" and losing focus in the moment. Much appreciated!

@jjngx jjngx merged commit e69b9bc into main Jul 12, 2023
@jjngx jjngx deleted the jputrino-patch-1 branch July 12, 2023 16:21
ADubhlaoich added a commit that referenced this pull request Jul 13, 2023
Fixes a typo in the GlobalConfiguration CRD step.

Signed-off-by: Jodie Putrino <[email protected]>
Co-authored-by: Alan Dooley <[email protected]>
jjngx pushed a commit that referenced this pull request Jul 13, 2023
…#4097)

This commit includes two small changes: the first is removing a line
from the Hugo configuration file that was preventing the appropriate
styling to occur for code blocks. The second is to fix an incorrectly
closed table in the technical specifications page.

The largest change is reformatting of the Installation with Manifests
page. Multiple sections of the page now have horizontal lines to
separate them visually: many instances of notes were replaced with the
formal Hugo shortcode for notes. Indentation was also fixed in multiple
places, with subheadings being used to create explicit flow where
previously it was implicit.

Finally, there are multiple instances of nouns being fixed, as well as
certain portions of text being rephrased for clarity or conciseness.

Signed-off-by: Jodie Putrino <[email protected]>

---------

Signed-off-by: Jodie Putrino <[email protected]>
Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
Co-authored-by: Jodie Putrino <[email protected]>

* Update installation-with-manifests.md (#4090)

Fixes a typo in the GlobalConfiguration CRD step.

Signed-off-by: Jodie Putrino <[email protected]>
Co-authored-by: Alan Dooley <[email protected]>

---------

Signed-off-by: Jodie Putrino <[email protected]>
Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
Co-authored-by: Jodie Putrino <[email protected]>
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
None yet
Development

Successfully merging this pull request may close these issues.

6 participants