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

Add codespell configuration and workflow (just to check, not fix) #4692

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

yarikoptic
Copy link

Pull Request

Related to all the commits which fixed typos in the past, e.g.

❯ git lg | grep typo | nl | tail
    37	| | * | b8cd75cb - Fixed typo in pipeline (7 years ago) [aquilent-robproper]
    38	| | * | c519ed0d - Fixed typo in pipeline (7 years ago) [aquilent-robproper]
    39	| | * | | 1b2ad00a - Fixed typo in Jenkins pipeline (7 years ago) [aquilent-robproper]
    40	| | * | | 2a882d36 - Fixed annotation typo in Jenkins pipeline (7 years ago) [aquilent-robproper]
    41	| | * | | 77d1482d - Fixed typo in Jenkins pipeline (7 years ago) [aquilent-robproper]
    42	| * | | | | bed33b8d - Fix small typo in catalog-web-source.j2 (7 years ago) [Dan Olaru]
    43	| * | b57160e3 - typo (7 years ago) [Alex Perfilov]
    44	| * c3d8dc37 - small typo (7 years ago) [Dan Olaru]
    45	* 48b422c2 - typo fix (8 years ago) [fxia]
    46	* | cc638dec - typo fix (8 years ago) [fxia]

About

just adds basic configuration and action to do codespell on the codebase. ATM seems to be typos free, congrats!

feel free to close... just was around and thus decided to contribute

PR TASKS

  • The actual code changes.
  • Tests written and passed.
  • Any changes to docs?
  • Any dependent PRs needed?

Copy link

@xahmedx777x xahmedx777x left a comment

Choose a reason for hiding this comment

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

Ax

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.

None yet

3 participants