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

✏️ Documentation spelling fixes #4497

Merged
merged 1 commit into from
Apr 24, 2022

Conversation

richardsondev
Copy link
Contributor

  • Your changes are not possible to do through a plugin and relevant
    to a large audience (ideally all users of OctoPrint)
  • If your changes are large or otherwise disruptive: You have
    made sure your changes don't interfere with current development by
    talking it through with the maintainers, e.g. through a
    Brainstorming ticket
  • Your PR targets OctoPrint's devel branch if it's a completely
    new feature, or maintenance if it's a bug fix or improvement of
    existing functionality for the current stable version (no PRs
    against master or anything else please)
  • Your PR was opened from a custom branch on your repository
    (no PRs from your version of master, maintenance or devel please),
    e.g. dev/my_new_feature or fix/my_bugfix
  • Your PR only contains relevant changes: no unrelated files,
    no dead code, ideally only one commit - rebase and squash your PR
    if necessary!
  • Your changes follow the existing coding style
  • If your changes include style sheets: You have modified the
    .less source files, not the .css files (those are generated with
    lessc)
  • You have tested your changes (please state how!) - ideally you
    have added unit tests
  • You have run the existing unit tests against your changes and
    nothing broke
  • You have added yourself to the AUTHORS.md file :)

What does this PR do and why is it necessary?

Fixes minor spelling issues in the documentation across several areas. Since there were only documentation spelling changes, I did not run any unit tests aside from the pre-commit checks.

How was it tested? How can it be tested by the reviewer?

I verified each word changed using the Cambridge Dictionary to ensure the spelling is now correct.

Any background context you want to provide?

N/A

What are the relevant tickets if any?

N/A

Screenshots (if appropriate)

N/A

Further notes

N/A

@github-actions github-actions bot added docs Related to documentation targets maintenance The PR targets the maintenance branch approved Issue has been approved by the bot or manually for further processing labels Apr 23, 2022
@cp2004 cp2004 force-pushed the jorichar/docspelling branch from a5ccc6b to 9b442fe Compare April 24, 2022 16:05
@cp2004 cp2004 changed the base branch from maintenance to master April 24, 2022 16:06
@cp2004
Copy link
Member

cp2004 commented Apr 24, 2022

Switched the target to master, since the only edits were to docs rst files it can go there to be deployed straight away.

@cp2004 cp2004 removed the targets maintenance The PR targets the maintenance branch label Apr 24, 2022
@cp2004 cp2004 merged commit 64c21ba into OctoPrint:master Apr 24, 2022
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Apr 25, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
approved Issue has been approved by the bot or manually for further processing docs Related to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants