Merged
Conversation
Codecov Report
|
jsoref
commented
Dec 29, 2019
examples/jib-multimodule/.mvn/wrapper/MavenWrapperDownloader.java
Outdated
Show resolved
Hide resolved
dgageot
suggested changes
Dec 31, 2019
Contributor
dgageot
left a comment
There was a problem hiding this comment.
It's awesome. Just a few 3rd party files that I'd rather we don't edit.
examples/jib-multimodule/.mvn/wrapper/MavenWrapperDownloader.java
Outdated
Show resolved
Hide resolved
Contributor
Author
|
Hmm, somehow, I think I lost some commits. Please bear with me. |
Contributor
Author
|
@dgageot: I've added those files to my ignores and updated the PR. |
dgageot
approved these changes
Jan 1, 2020
Contributor
|
@jsoref That's awesome. I'd love to run the check on each PR. Isn't that too complicated? |
Contributor
Author
|
Not terribly: I'm hoping to write a version that stores the word list in a bucket for our own use... |
Contributor
|
This is so cool :D Thank you! |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Description
Generated by https://github.com/jsoref/spelling
f; to maintain your repo, please considerfchurnUser facing changes
I don't think anything user-visible has changed, but I'm not absolutely certain.
After
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
Reviewer Notes