Skip to content

Fixing Typos#329

Open
Clarkcj12 wants to merge 4 commits into
pterodactyl:developfrom
Clarkcj12:develop
Open

Fixing Typos#329
Clarkcj12 wants to merge 4 commits into
pterodactyl:developfrom
Clarkcj12:develop

Conversation

@Clarkcj12

Copy link
Copy Markdown

Easy fix just fixed some typos I found in comments.

Copilot AI review requested due to automatic review settings June 17, 2026 16:29

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Note

Copilot was unable to run its full agentic suite in this review.

This PR cleans up typos in tests, comments, and configuration documentation to improve readability and clarity.

Changes:

  • Fixes spelling in a test case description for NewAtomicBool.
  • Corrects typos in middleware and Docker environment comments.
  • Updates config documentation to use the correct spelling for “usage”.

Reviewed changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated no comments.

File Description
system/utils_test.go Fixes typo in test description string.
router/middleware/middleware.go Fixes typo in an authorization middleware comment.
environment/docker/power.go Fixes typo in a Docker stop-path comment.
config/config.go Fixes typo in backup compression documentation comment.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@robertdrakedennis

Copy link
Copy Markdown
Contributor

when did copilot get enabled wtf

@Clarkcj12

Copy link
Copy Markdown
Author

No clue, I have no idea why it says I requested review when I didn't. Might have been a glitch with GitHub.

@simonmicro

simonmicro commented Jun 27, 2026

Copy link
Copy Markdown

Heyho, got another typo in the log messages:

s.Log().Error("could create base environment for server...")

...even worse, the actual causing error is also not logged

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants