Skip to content

Add codespell support (config, workflow to detect/not fix) and make it fix few typos#71

Open
yarikoptic wants to merge 3 commits intoProryanator:mainfrom
yarikoptic:enh-codespell
Open

Add codespell support (config, workflow to detect/not fix) and make it fix few typos#71
yarikoptic wants to merge 3 commits intoProryanator:mainfrom
yarikoptic:enh-codespell

Conversation

@yarikoptic
Copy link

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.

…s automagically

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

Thank you for opening this PR! Lemme try to get the pipeline to run and take a look at how this works 👏 (would like to loosen up the repo rules to not require my permission for running unit tests or pipelines like these).

@yarikoptic
Copy link
Author

odd... not that much was ran overall. Some times github CI makes me wonder ;-)

@Proryanator
Copy link
Owner

odd... not that much was ran overall. Some times github CI makes me wonder ;-)

Me too me too, I'm a Gitlab CI kind of guy at work 😆

@yarikoptic
Copy link
Author

came back to this just by chance (was doing benchmarking of qr decoding with claude - if of any interest https://github.com/ReproNim/qr_decode_bm)! should this be merged (or closed) and be done with it? ;-)

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