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 spelling #12040

Merged
merged 52 commits into from
May 23, 2022
Merged

Fix spelling #12040

merged 52 commits into from
May 23, 2022

Conversation

jsoref
Copy link
Contributor

@jsoref jsoref commented May 1, 2022

This PR corrects misspellings identified by the check-spelling action.

The misspellings have been reported at jsoref@c8485a3#commitcomment-72631828

The action reports that the changes in this PR would make it happy: jsoref@a086ddd

Note: this PR does not include the action. If you're interested in running a spell check on every PR and push, that can be offered separately.

Copy link
Contributor Author

@jsoref jsoref left a comment

Choose a reason for hiding this comment

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

Most corrections were automatically suggested by Google Sheets.
Duplicate word handling is entirely me. All fault is mine.

* `Time` now honours Dayling Saving and `ENV["TZ"]`
* `Time` now honours Daylight Saving and `ENV["TZ"]`
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Notable change.

Some projects don't like retroactively changing changelogs, if this is one, just let me know.

Copy link
Member

Choose a reason for hiding this comment

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

I prefer fixing such typos in the changelog. Thanks!

* `HTML`: `HTLM.escape` includes more characters (thanks @Ryuuzakis)
* `HTML`: `HTML.escape` includes more characters (thanks @Ryuuzakis)
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Notable

Label `in-progress` is used to signal that someone from the community is already working on the issue (since Github does not allow for a non-team member to be _assigned_ to an issue).
Label `in-progress` is used to signal that someone from the community is already working on the issue (since GitHub does not allow for a non-team member to be _assigned_ to an issue).
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Brand

lib/markd/README.md Outdated Show resolved Hide resolved
spec/compiler/codegen/class_var_spec.cr Outdated Show resolved Hide resolved
src/compiler/crystal/tools/playground/public/settings.js Outdated Show resolved Hide resolved
.append(this.modalContenDom = $("<div>").addClass("modal-content"))
.append(this.modalContentDom = $("<div>").addClass("modal-content"))
Copy link
Contributor Author

Choose a reason for hiding this comment

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

notable

src/crystal/dwarf/abbrev.cr Show resolved Hide resolved
# Executes a request request and yields an `HTTP::Client::Response` to the block.
# Executes a request and yields an `HTTP::Client::Response` to the block.
Copy link
Contributor Author

Choose a reason for hiding this comment

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

?

# "!#$%&'*+-.0123456789ABCDEFGHIJKLMNOPQRSTUWVXYZ^_`abcdefghijklmnopqrstuvwxyz|~"
# "!#$%&'*+-.0123456789ABCDEFGHIJKLMNOPQRSTUVWXYZ^_`abcdefghijklmnopqrstuvwxyz|~"
Copy link
Contributor Author

Choose a reason for hiding this comment

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

-UWVX
+UVWX

lib/markd/README.md Outdated Show resolved Hide resolved
@Blacksmoke16 Blacksmoke16 added kind:bug A bug in the code. Does not apply to documentation, specs, etc. kind:docs labels May 2, 2022
jsoref added 4 commits May 1, 2022 21:37
Signed-off-by: Josh Soref <[email protected]>
Signed-off-by: Josh Soref <[email protected]>
Signed-off-by: Josh Soref <[email protected]>
@straight-shoota straight-shoota self-requested a review May 2, 2022 22:36
Copy link
Member

@straight-shoota straight-shoota left a comment

Choose a reason for hiding this comment

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

Thank you very much for this great work! It's nice to see that you didn't just run some automation over the code base but vetted every single change it suggested.

* `Time` now honours Dayling Saving and `ENV["TZ"]`
* `Time` now honours Daylight Saving and `ENV["TZ"]`
Copy link
Member

Choose a reason for hiding this comment

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

I prefer fixing such typos in the changelog. Thanks!

spec/compiler/semantic/if_spec.cr Outdated Show resolved Hide resolved
spec/compiler/semantic/while_spec.cr Show resolved Hide resolved
src/compiler/crystal/tools/playground/public/settings.js Outdated Show resolved Hide resolved
src/crystal/dwarf/abbrev.cr Show resolved Hide resolved
src/io/overlapped.cr Outdated Show resolved Hide resolved
jsoref added 16 commits May 3, 2022 12:58
Signed-off-by: Josh Soref <[email protected]>
Signed-off-by: Josh Soref <[email protected]>
Signed-off-by: Josh Soref <[email protected]>
Signed-off-by: Josh Soref <[email protected]>
Signed-off-by: Josh Soref <[email protected]>
Signed-off-by: Josh Soref <[email protected]>
Signed-off-by: Josh Soref <[email protected]>
Signed-off-by: Josh Soref <[email protected]>
Signed-off-by: Josh Soref <[email protected]>
Signed-off-by: Josh Soref <[email protected]>
Signed-off-by: Josh Soref <[email protected]>
Signed-off-by: Josh Soref <[email protected]>
Signed-off-by: Josh Soref <[email protected]>
Signed-off-by: Josh Soref <[email protected]>
Signed-off-by: Josh Soref <[email protected]>
Signed-off-by: Josh Soref <[email protected]>
jsoref added 20 commits May 3, 2022 12:58
Signed-off-by: Josh Soref <[email protected]>
Signed-off-by: Josh Soref <[email protected]>
Signed-off-by: Josh Soref <[email protected]>
Signed-off-by: Josh Soref <[email protected]>
Signed-off-by: Josh Soref <[email protected]>
Signed-off-by: Josh Soref <[email protected]>
Signed-off-by: Josh Soref <[email protected]>
Signed-off-by: Josh Soref <[email protected]>
Signed-off-by: Josh Soref <[email protected]>
Signed-off-by: Josh Soref <[email protected]>
Signed-off-by: Josh Soref <[email protected]>
Signed-off-by: Josh Soref <[email protected]>
Signed-off-by: Josh Soref <[email protected]>
Signed-off-by: Josh Soref <[email protected]>
Signed-off-by: Josh Soref <[email protected]>
Signed-off-by: Josh Soref <[email protected]>
Signed-off-by: Josh Soref <[email protected]>
Signed-off-by: Josh Soref <[email protected]>
Signed-off-by: Josh Soref <[email protected]>
Signed-off-by: Josh Soref <[email protected]>
Copy link
Member

@beta-ziliani beta-ziliani left a comment

Choose a reason for hiding this comment

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

🚀 thanks!

@beta-ziliani beta-ziliani added this to the 1.5.0 milestone May 20, 2022
@straight-shoota straight-shoota changed the title Spelling Fix spelling May 23, 2022
@straight-shoota straight-shoota merged commit 04f1c5a into crystal-lang:master May 23, 2022
@jsoref jsoref deleted the spelling branch May 23, 2022 12:47
HertzDevil added a commit to HertzDevil/crystal that referenced this pull request Jul 2, 2022
This instance of the spelling mistake was not detected by crystal-lang#12040 for some reason, so that PR broke the playground when something fails to build.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind:bug A bug in the code. Does not apply to documentation, specs, etc. kind:docs
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants