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 #1738

Merged
merged 24 commits into from
Jun 13, 2023
Merged

Spelling #1738

merged 24 commits into from
Jun 13, 2023

Conversation

jsoref
Copy link
Contributor

@jsoref jsoref commented Jun 13, 2023

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

The misspellings have been reported at https://github.com/jsoref/atlas/actions/runs/5254190298#summary-14217513198

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

jsoref added 23 commits June 13, 2023 02:53
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>
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>
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>
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>
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>
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>
- name: Verify generated files are checked in properly
- name: Verify generated files are checked-in properly
Copy link
Contributor Author

Choose a reason for hiding this comment

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

This form is already in use in this repository. I think the way it's being used today isn't technically correct, but this matches that style.

If you're actually picky about checked in, those other instances should probably be changed...

doc/md/cloud/getting-started.md Outdated Show resolved Hide resolved
After logging in (not including the first login), you will see all the organizations you have access to. Choose the organization you would to log in to.
After logging in (not including the first login), you will see all the organizations you have access to. Choose the organization you would to log into.
Copy link
Contributor Author

Choose a reason for hiding this comment

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

to really shouldn't be left dangling here.

$ export REPO="username/repo_name" # e.g. "arigia/atlas"
$ export REPO="username/repo_name" # e.g. "ariga/atlas"
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?

and [Fly](https://fly.io/docs/app-guides/continuous-deployment-with-github-actions/) Github actions. For additional
and [Fly](https://fly.io/docs/app-guides/continuous-deployment-with-github-actions/) GitHub Actions. For additional
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

scorecard((science + mathamatics)/2);
scorecard((science + mathematics)/2);
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 assuming this is meant to be a word...

### Implementing Virtual Generated Columns Columns with Atlas
### Implementing Virtual Generated Columns with Atlas
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 assuming this was an unintentionally duplicated word, but I could be wrong....

@a8m a8m self-requested a review June 13, 2023 20:13
@a8m
Copy link
Member

a8m commented Jun 13, 2023

Thanks a lot for the contribution, @jsoref. It's highly appreciated.

I'll merge once CI is green

@a8m a8m merged commit 661ade0 into ariga:master Jun 13, 2023
25 checks passed
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.

None yet

2 participants