-
Notifications
You must be signed in to change notification settings - Fork 378
Spelling #207
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 #207
Conversation
Thanks for the changes. Looks like you need to sign your commits before we can merge. Can you please setup commit signing and re-push your commits? |
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]>
@adityaramani done |
...
why does it say 1 issue when two tests failed? (I'm assuming 39 didn't fail...) |
I think it was just the one test that failed. There were two log line recorded since one was for the actual test, and other saying that a test group failed. Re-ran them and looks like it was a flake |
This PR corrects misspellings identified by the check-spelling action
The misspellings have been reported at https://github.com/jsoref/container/actions/runs/15662939575/attempts/1
The action reports that the changes in this PR would make it happy: https://github.com/jsoref/container/actions/runs/15662939742/attempts/1#summary-44123289718