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

Fix typos in console and tests #436

Merged
merged 2 commits into from
Feb 13, 2023
Merged

Conversation

yarikoptic
Copy link
Contributor

No description provided.

@dbieber
Copy link
Collaborator

dbieber commented Feb 13, 2023

Thanks for fixing the typos. Let's not add codespell at this time, but I'd be happy to merge the PR w/ just the typo fixes.

@yarikoptic
Copy link
Contributor Author

You are the boss. I force pushed kicking away the last commit with the workflow. Let me know if you would like to get rid of .codespellrc as well

@dbieber
Copy link
Collaborator

dbieber commented Feb 13, 2023

Let me know if you would like to get rid of .codespellrc as well

Yep, thanks

=== Do not change lines below ===
{
 "chain": [],
 "cmd": "git-sedi thats 'that is'",
 "exit": 0,
 "extra_inputs": [],
 "inputs": [],
 "outputs": [],
 "pwd": "."
}
^^^ Do not change lines above ^^^
@yarikoptic
Copy link
Contributor Author

IMHO "your loss", but done now.

@dbieber
Copy link
Collaborator

dbieber commented Feb 13, 2023

Thanks Yari! I'm not opposed to adding spell check more generally; just adding codespell would require more research and review on my part and would delay getting things checked in. So, I'm happy to merge the pure-fix PR now rather than having an easy fix get blocked by a slower process.

@dbieber dbieber changed the title Codespell fix typos, add workflow for the future to catch new ones Fix typos in console and tests Feb 13, 2023
@dbieber dbieber merged commit 527e3d0 into google:master Feb 13, 2023
@yarikoptic
Copy link
Contributor Author

Understood. Let me know if your want sister pr with those

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