various typos + typos configuration + CI #5720
Conversation
phip1611
force-pushed
the
typos
branch
4 times, most recently
from
51ea0c0
to
2e2c029
Compare
.typos.toml
Outdated
| Ded = "Ded" # GUIDed | ||
| ba = "ba" | ||
| Cros = "Cros" # CrosVM | ||
| Dake = "Dake" |
There was a problem hiding this comment.
I wonder what's special about that name and not any of the others?
There was a problem hiding this comment.
You would think the tool would be able to identify that pretty common email format (Name Name <email>) and not flag it
There was a problem hiding this comment.
Agree, unfortunately it does not.
phip1611
force-pushed
the
typos
branch
3 times, most recently
from
0185cee
to
6365684
Compare
There was a problem hiding this comment.
Thank you for the contributions. Generally look good to me. Couple of cosmetic comments about the commit style. Please refer our contribution doc for complete details [1].
- Please merge the last commit about crosvm to the fist commit;
- Please rename the first commit to:
misc: Fix various typos; - Please capitalize the first world for each commit subject after the component name.
[1] https://github.com/cloud-hypervisor/cloud-hypervisor/blob/main/CONTRIBUTING.md
I updated the PR. I hope, we are good to go now. If not, let me know. |
|
Please rebase. |
This is a preparation to run the typos utility on the repo. Signed-off-by: Philipp Schuster <philipp.schuster@cyberus-technology.de>
This fixes all typos found by the typos utility with respect to the config file. Signed-off-by: Philipp Schuster <philipp.schuster@cyberus-technology.de>
This adds the typos utility as dedicated CI step. Signed-off-by: Philipp Schuster <philipp.schuster@cyberus-technology.de>
Done. |
likebreath
added
the
CI
This fixes all typos I've found using the https://github.com/crate-ci/typos utility.
Furthermore, it adds an additional step to execute
typosin GitHub CI.