Skip to content

Add codespell support (config, workflow to detect/not fix) and make it fix a typo#458

Merged
Wendong-Fan merged 5 commits intocamel-ai:mainfrom
yarikoptic:enh-codespell
Apr 3, 2025
Merged

Add codespell support (config, workflow to detect/not fix) and make it fix a typo#458
Wendong-Fan merged 5 commits intocamel-ai:mainfrom
yarikoptic:enh-codespell

Conversation

@yarikoptic
Copy link
Contributor

More about codespell: https://github.com/codespell-project/codespell .

I personally introduced it to dozens if not hundreds of projects already and so far only positive feedback.

CI workflow has 'permissions' set only to 'read' so also should be safe.

Although currently only 1 typo found/fixed, fixing typos has been very popular recently ;)

❯ git lg origin/main  | grep typo
* | | 3140c05 - fix typo (11 days ago) [Wendong]
*   86e03a1 - Fixing typo visible in webapp.py: fresh re-iteration of https://github.com/camel-ai/owl/pull/331 (#380) (12 days ago) [Wendong-Fan]
| * 171c945 - Fixing typo visible in webapp.py (13 days ago) [Didier Durand]
* | 1da3805 - fix readme typo (3 weeks ago) [Wendong]
* da096d1 - fix typo (4 weeks ago) [lazychih114]
* 2436c21 - fix typo (4 weeks ago) [lazychih114]

so with codespell it might become needed less

…agically

=== Do not change lines below ===
{
 "chain": [],
 "cmd": "codespell -w",
 "exit": 0,
 "extra_inputs": [],
 "inputs": [],
 "outputs": [],
 "pwd": "."
}
^^^ Do not change lines above ^^^
Copy link
Member

@Wendong-Fan Wendong-Fan left a comment

Choose a reason for hiding this comment

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

thanks @yarikoptic !

@Wendong-Fan Wendong-Fan merged commit 5408dcd into camel-ai:main Apr 3, 2025
1 check passed
jairodriguez pushed a commit to jairodriguez/autonomica that referenced this pull request Jun 19, 2025
jairodriguez pushed a commit to jairodriguez/autonomica that referenced this pull request Jun 21, 2025
LJPearson176 pushed a commit to LJPearson176/hyperion that referenced this pull request Aug 7, 2025
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