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

chore: fix spelling #2073

Closed
wants to merge 37 commits into from
Closed

chore: fix spelling #2073

wants to merge 37 commits into from

Conversation

jsoref
Copy link

@jsoref jsoref commented Oct 4, 2022

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

The misspellings have been reported at jsoref@bb80c9f#commitcomment-85842238

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

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.

jsoref added 30 commits October 4, 2022 16:35
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]>
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 7 commits October 4, 2022 16:40
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 jsoref requested a review from a team as a code owner October 4, 2022 20:46
Copy link
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 changes were suggested by Google Sheets. All fault mine.

I'll respond to comments on Thursday or sometime late next week.

Note that I leave commits initially as individual word families as it makes it easier to drop objectionable items. I'm happy to squash once people are happy w/ the general changes.

I'm also happy to reword the commits if necessary. But it looks like the requirement is only on the PR for conventional-commits and not the individual items, so if I can avoid touching commits until the end, that'd be great (and I'd rather squash than adjust each commit message to chore(...): ...).

log in to Heroku Container Registry
log into Heroku Container Registry
Copy link
Author

Choose a reason for hiding this comment

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

I'm currently experimenting with in to/into. People have strong opinions about it. I'm happy to drop this or any other change. (Note that I didn't change all in tos to into.)

My claim is that the verb is log and the preposition is into. But others appear to disagree.

it('has a missing missing output', function () {
it('has a missing output', function () {
Copy link
Author

Choose a reason for hiding this comment

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

It's possible this really is talking about missing a missing output. But I'm guessing it isn't. (This is the second most recent thing w/ which I'm experimenting as analysis items go...)

// --app=my-app is consided a flag & not a flag value
// --app=my-app is considered a flag & not a flag value
Copy link
Author

Choose a reason for hiding this comment

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

I didn't pick this. I blame Google Sheets (but I do think it's correct).

- Releax engines requirements in package.json.
- Relax engines requirements in package.json.
Copy link
Author

Choose a reason for hiding this comment

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

So projects don't like changing changelogs.

It's also possible some files come from an upstream. In general, I'm happy to try to send changes upstream, but I don't make promises as to when (I have a long queue). Happy to drop if there's an upstream...

* Pipeline repositories link a pipeline to a Github repository.
* Pipeline repositories link a pipeline to a GitHub repository.
Copy link
Author

Choose a reason for hiding this comment

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

brand

{key: 'iconStatus', width: 1, label: ''}, // label '' is to make sure that widh is 1 character
{key: 'iconStatus', width: 1, label: ''}, // label '' is to make sure that width is 1 character
Copy link
Author

Choose a reason for hiding this comment

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

Amusingly, Google Sheets guessed wish...

// we require *an* add-on to interact with, make sure that that add-on
// we require *an* add-on to interact with, make sure that add-on
Copy link
Author

Choose a reason for hiding this comment

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

that that is more often correct vs the average doubled word. But I don't think it's necessary here.

const { formation, metaas_source, port, ...filteredRedis } = redis
const { formation, meta_as_source, port, ...filteredRedis } = redis
Copy link
Author

Choose a reason for hiding this comment

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

Not sure what this is doing.
https://cs.github.com/?scopeName=All+repos&scope=&q=metaas_source
shows 5 hits that more or less link back to this thing. It's probably an API.

I left it here so that I could ask "what is this?"

echo 'HERERE'
echo 'HERE'
Copy link
Author

Choose a reason for hiding this comment

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

My favorite item. This came w/ the original block. Does it need to be here? :)

@eablack eablack mentioned this pull request Mar 7, 2024
@eablack
Copy link
Contributor

eablack commented Mar 7, 2024

This is old enough that it's probably too much work to rebase, so I went ahead and recreated the changes here: #2694 thanks @jsoref !

@eablack eablack closed this Mar 7, 2024
@jsoref jsoref deleted the spelling branch March 7, 2024 21:21
@jsoref jsoref mentioned this pull request Mar 8, 2024
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