Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Fix typos #707

Merged
merged 1 commit into from
Apr 3, 2024
Merged

Fix typos #707

merged 1 commit into from
Apr 3, 2024

Conversation

kianmeng
Copy link
Contributor

@kianmeng kianmeng commented Apr 2, 2024

Found via typos --format brief

Copy link
Member

@geemus geemus left a comment

Choose a reason for hiding this comment

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

Thanks for all the fixes. I think one of the corrections was inaccurate, but otherwise looking good.

@@ -19,7 +19,7 @@ class Real
# * :engine_version <~String> - The Cluster's caching software version
# * :notification_topic_arn <~String> - Amazon SNS Resource Name
# * :port <~Integer> - The memcached port number
# * :preferred_availablility_zone <~String>
# * :preferred_availabilility_zone <~String>
Copy link
Member

Choose a reason for hiding this comment

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

This correction looks off, I think it needs one fewer il in the middle? That should match with the other change below and the related one in the other file above.

Found via `typos --format brief`
Copy link
Member

@geemus geemus left a comment

Choose a reason for hiding this comment

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

Thanks!

@geemus geemus merged commit 7053afa into fog:master Apr 3, 2024
8 checks passed
@kianmeng
Copy link
Contributor Author

kianmeng commented Apr 4, 2024

🥳 🥳 🥳 🥳 🥳

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants