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

Fix typos #566

Merged
merged 1 commit into from
Apr 26, 2024
Merged

Fix typos #566

merged 1 commit into from
Apr 26, 2024

Conversation

kianmeng
Copy link
Contributor

Found via codespell -L te,wronly,noes,nulll and typos --format brief

@@ -207,12 +207,12 @@ def register_type(format, name, encoder_class, decoder_class)

# Alias the +old+ type to the +new+ type.
def alias_type(format, new, old)
[:encoder, :decoder].each do |ende|
Copy link
Collaborator

Choose a reason for hiding this comment

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

ende means encoder-decoder in this context. It is no spelling mistake.

@larskanis
Copy link
Collaborator

Thank you very much for scanning and correction! It's all OK with the exception of ende->end.

Found via `codespell -L te,wronly,noes,nulll,ende` and `typos --format brief`
@kianmeng kianmeng requested a review from larskanis April 26, 2024 07:19
@larskanis larskanis merged commit 897042a into ged:master Apr 26, 2024
8 of 14 checks passed
@larskanis
Copy link
Collaborator

Nice - thank you!

@@ -543,7 +543,7 @@ Init_pg_ext(void)
/* Result#result_error_field argument constant
*
* The SQLSTATE code for the error.
* The SQLSTATE code identies the type of error that has occurred; it can be used by front-end applications to perform specific operations (such as error handling) in response to a particular database error.
* The SQLSTATE code identities the type of error that has occurred; it can be used by front-end applications to perform specific operations (such as error handling) in response to a particular database error.
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggest: identifies

Copy link
Collaborator

Choose a reason for hiding this comment

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

Agreed.

@@ -171,7 +171,7 @@ module Checker
include Checker

def initialize
# The key of these hashs maps to the `typname` column from the table pg_type.
# The key of these hashes maps to the `typname` column from the table pg_type.
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggest: hash

Copy link
Collaborator

Choose a reason for hiding this comment

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

I opened #567 to better describe the content of @coders_by_name. Is it OK?

larskanis added a commit to larskanis/ruby-pg that referenced this pull request Apr 26, 2024
@kianmeng
Copy link
Contributor Author

🥳 🥳 🥳 🥳 🥳

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