-
Notifications
You must be signed in to change notification settings - Fork 227
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
chore: fix spelling #2073
Conversation
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]>
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]>
There was a problem hiding this 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 |
There was a problem hiding this comment.
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 to
s 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 () { |
There was a problem hiding this comment.
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 |
There was a problem hiding this comment.
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. |
There was a problem hiding this comment.
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. |
There was a problem hiding this comment.
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 |
There was a problem hiding this comment.
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 |
There was a problem hiding this comment.
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 |
There was a problem hiding this comment.
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' |
There was a problem hiding this comment.
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? :)
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.