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.

Sign up for GitHub

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

Jump to bottom

Fix some documentation typos & grammar #285

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

Fix some documentation typos & grammar #285

wants to merge 2 commits into from

Conversation

trinitronx
Copy link

Fixing some documentation typos + grammar:

  • Fix Chef Effortless doc typo: "help with you with" => "help you with"
  • Ran paragraph through Grammarly:
    • Simplified repeated "better manage"
    • Adjusted comma usage
    • Removed 'to' articles
    • Feel free to revert this commit if you don't like what it did.

@trinitronx trinitronx requested a review from a team as a code owner January 11, 2022 01:49
@chef-expeditor
Copy link
Contributor

Hello trinitronx! Thanks for the pull request!

Here is what will happen next:

  1. Your PR will be reviewed by the maintainers.
  2. If everything looks good, one of them will approve it, and your PR will be merged.

Thank you for contributing!

@trinitronx
Ran paragraph through Grammarly
1548b1f 
Looks like it simplified repeated "better manage",
adjusted comma usage, and removed 'to' articles

Feel free to revert this commit if you don't like what it did.

Signed-off-by: James Cuzella <[email protected]>
@trinitronx
Copy link
Author

Looks like the CI failure is from GitHub's recent disabling of git:// protocol.


fatal: remote error:
      The unauthenticated git protocol on port 9418 is no longer supported.
    Please see https://github.blog/2021-09-01-improving-git-protocol-security-github/ for more information

Seems reproducible locally with pre-commit

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@rahulgoel1 rahulgoel1 rahulgoel1 approved these changes

@IanMadd IanMadd IanMadd approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@trinitronx @IanMadd @rahulgoel1