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

Spelling #65

Merged
merged 7 commits into from
Jul 14, 2023
Merged

Spelling #65

merged 7 commits into from
Jul 14, 2023

Conversation

jsoref
Copy link
Contributor

@jsoref jsoref commented Jun 14, 2023

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

The misspellings have been reported at https://github.com/jsoref/orb/actions/runs/5270930886#summary-14267025062

The action reports that the changes in this PR would make it happy: https://github.com/jsoref/orb/actions/runs/5270931012#summary-14267025554

jsoref added 3 commits June 13, 2023 20:08
Signed-off-by: Josh Soref <2119212+jsoref@users.noreply.github.com>
Signed-off-by: Josh Soref <2119212+jsoref@users.noreply.github.com>
Signed-off-by: Josh Soref <2119212+jsoref@users.noreply.github.com>
@jsoref jsoref requested review from cizl and tonilastre as code owners June 14, 2023 18:43
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 automatically suggested by Google Sheets. All fault mine

free to check other Javascript examples in `examples/` directory.
free to check other JavaScript examples in `examples/` directory.
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

const getPointBrezier = <N extends INodeBase, E extends IEdgeBase>(
const getPointBezier = <N extends INodeBase, E extends IEdgeBase>(
Copy link
Contributor Author

Choose a reason for hiding this comment

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

I'm hoping I'm right...

src/utils/entity.utils.ts Outdated Show resolved Hide resolved
@tonilastre tonilastre changed the base branch from main to release/0.5.0 July 13, 2023 08:23
jsoref added 4 commits July 13, 2023 10:50
Signed-off-by: Josh Soref <2119212+jsoref@users.noreply.github.com>
Signed-off-by: Josh Soref <2119212+jsoref@users.noreply.github.com>
Signed-off-by: Josh Soref <2119212+jsoref@users.noreply.github.com>
Signed-off-by: Josh Soref <2119212+jsoref@users.noreply.github.com>
Copy link
Contributor

@tonilastre tonilastre left a comment

Choose a reason for hiding this comment

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

Thanks @jsoref!

Btw we will also add your action as part of the build/verify process. This is really cool stuff!

@tonilastre tonilastre merged commit 202c945 into memgraph:release/0.5.0 Jul 14, 2023
@jsoref jsoref deleted the spelling branch July 14, 2023 14:27
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.

2 participants