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

Merged
merged 38 commits into from
Jan 2, 2020
Merged

Spelling #3458

merged 38 commits into from
Jan 2, 2020

Conversation

jsoref
Copy link
Contributor

@jsoref jsoref commented Dec 29, 2019

Description

Generated by https://github.com/jsoref/spelling f; to maintain your repo, please consider fchurn

  • I've selected en-US over en-GB as the majority of the words appear to favor that (especially for the handful of en-GB words I encountered) -- I can include a marker in the commit message for such words if necessary, but I haven't here.
  • I try to avoid changes files that appear to be from third-party code, but I'm a (sleepy) human, if I missed some code, please let me know, I'll try to apply the same algorithm to the upstream (and drop the commits locally).

User facing changes

I don't think anything user-visible has changed, but I'm not absolutely certain.

After

If behavior changes: describe succintly the behavior after your change

Ok, this specific line in future PRs will be changed.

Next PRs.

I might offer a PR to remove misspellings from CHANGELOG.md. I know it's nice to record all changes in the changelog, but it's really frustrating when running a spelling-checker to have to actively say "oh, yes, they intentionally misspelled that word to say that they used to misspell that word". It's better for commits to only mention the correct spelling, if someone is curious, they can use history to learn all of the misspellings.

Submitter Checklist

  • Includes unit tests
  • Mentions any output changes.
  • Adds documentation as needed: user docs, YAML reference, CLI reference.
  • Adds integration tests if needed.

Reviewer Notes

  • The code flow looks good.
  • Unit test added.
  • User facing changes look good.

@codecov
Copy link

codecov bot commented Dec 29, 2019

Codecov Report

Merging #3458 into master will not change coverage.
The diff coverage is n/a.

Impacted Files Coverage Δ
pkg/skaffold/docker/authenticators.go 0% <ø> (ø) ⬆️

.github/pull_request_template.md Show resolved Hide resolved
CHANGELOG.md Show resolved Hide resolved
CHANGELOG.md Show resolved Hide resolved
cmd/skaffold/app/flags/image_test.go Show resolved Hide resolved
docs/design_proposals/digest-tagger.md Show resolved Hide resolved
docs/static/scripts/iframeResizer.js Outdated Show resolved Hide resolved
pkg/skaffold/deploy/kustomize_test.go Show resolved Hide resolved
pkg/skaffold/docker/reference_test.go Show resolved Hide resolved
pkg/skaffold/schema/v1alpha5/upgrade_test.go Show resolved Hide resolved
Copy link
Contributor

@dgageot dgageot left a comment

Choose a reason for hiding this comment

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

It's awesome. Just a few 3rd party files that I'd rather we don't edit.

docs/static/scripts/iframeResizer.contentWindow.js Outdated Show resolved Hide resolved
docs/static/scripts/iframeResizer.js Outdated Show resolved Hide resolved
examples/jib/.mvn/wrapper/MavenWrapperDownloader.java Outdated Show resolved Hide resolved
@jsoref
Copy link
Contributor Author

jsoref commented Dec 31, 2019

Hmm, somehow, I think I lost some commits. Please bear with me.
-- Commits restored, but I apparently lost my original commits, so if I need to resequence, I'll probably be unhappy.

@jsoref
Copy link
Contributor Author

jsoref commented Dec 31, 2019

@dgageot: I've added those files to my ignores and updated the PR.

@dgageot dgageot added the kokoro:run runs the kokoro jobs on a PR label Jan 1, 2020
@kokoro-team kokoro-team removed the kokoro:run runs the kokoro jobs on a PR label Jan 1, 2020
@dgageot dgageot added the kokoro:run runs the kokoro jobs on a PR label Jan 1, 2020
@dgageot
Copy link
Contributor

dgageot commented Jan 1, 2020

@jsoref That's awesome. I'd love to run the check on each PR. Isn't that too complicated?

@dgageot dgageot self-assigned this Jan 1, 2020
@jsoref
Copy link
Contributor Author

jsoref commented Jan 1, 2020

Not terribly:
https://github.com/checkstyle/checkstyle/blob/master/.ci/test-spelling-unknown-words.sh

I'm hoping to write a version that stores the word list in a bucket for our own use...

@dgageot dgageot merged commit d0c9ced into GoogleContainerTools:master Jan 2, 2020
@balopat
Copy link
Contributor

balopat commented Jan 16, 2020

This is so cool :D Thank you!

@jsoref jsoref deleted the spelling branch January 17, 2020 00:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cla: yes kokoro:run runs the kokoro jobs on a PR
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants