Skip to content

Fix typos in usage message and comments across multiple files#225

Merged
rowanseymour merged 1 commit intonyaruka:mainfrom
alexandear-org:fix/typos
Apr 1, 2026
Merged

Fix typos in usage message and comments across multiple files#225
rowanseymour merged 1 commit intonyaruka:mainfrom
alexandear-org:fix/typos

Conversation

@alexandear
Copy link
Copy Markdown
Contributor

This PR fixes typos in the filename, comments, and test name.

Copilot AI review requested due to automatic review settings April 1, 2026 19:21
@rowanseymour rowanseymour merged commit 29eedcc into nyaruka:main Apr 1, 2026
6 checks passed
@alexandear alexandear deleted the fix/typos branch April 1, 2026 19:24
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR cleans up spelling/typos in user-facing CLI usage text, test naming, and inline comments, improving readability and consistency across the project.

Changes:

  • Fixes typos in comments in phonenumbers.go and insertablebuffer.go.
  • Corrects a test function name typo in phonenumbers_test.go.
  • Fixes a typo in the phoneparser CLI usage message and adds metadata.go embedding declarations.

Reviewed changes

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

Show a summary per file
File Description
phonenumbers.go Corrects spelling in internal comments.
phonenumbers_test.go Renames a test to fix a typo in the function name.
metadata.go Introduces embedded metadata declarations for packaged data files.
insertablebuffer.go Corrects spelling in a comment (noted remaining informal phrasing).
cmd/phoneparser/main.go Fixes a typo in the CLI usage string.

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

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.

3 participants